add noStackEnvs option

[dankahle]

Another pull as the last one's copy/paste/edit features messed up the diff. It kinda uglifies a simple module, just throwing it out there for an option. There is a beauty in specifying which envs to not show the stack for. The true/false option is much simpler for the module, I'll give you that, just that this was easier for the user of the module is all. Junks things up though.

[coveralls]

Coverage decreased (-66.67%) to 33.33% when pulling 3728116 on dankahle:nostackenvs into df765a5 on expressjs:master.

[dougwilson]

We will only accept the true/false implementation.

[jonathanong]

yeah this implementation is over complicated. and process.env.NODE_ENV.toLowerCase() is going to fail if NODE_ENV doesn't happen to be defined.

[dankahle]

No biggy, just wanted to throw it out there. Thanks for looking at it.

On Thu, Mar 26, 2015 at 1:59 PM, jongleberry [email protected]
wrote:

yeah this implementation is over complicated. and
process.env.NODE_ENV.toLowerCase() is going to fail if NODE_ENV doesn't
happen to be defined.

—
Reply to this email directly or view it on GitHub
#2 (comment)
.