cool, will change it.


On Thu, Apr 25, 2013 at 12:46 PM, Brian J Brennan
<notifications@github.com>wrote:

> In queue.js:
>
> > @@ -0,0 +1,12 @@
> > +const sqs = require('sqs');
> > +const config = require('config');
> > +
> > +module.exports = sqs({
>
> NIT++: It's a bit non-idiomatic to modify module.exports more than once.
> It might be better to create an object, modify it as you do, then assign it
> to module.exports at the end.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mozilla/aestimia/pull/7/files#r3962625>
> .
>



-- 
@chmcavoy <http://twitter.com/chmcavoy>
http://lonelylion.com