Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add option to silence errors #62

Merged
merged 1 commit into from
Apr 1, 2015
Merged

add option to silence errors #62

merged 1 commit into from
Apr 1, 2015

Conversation

maxbeatty
Copy link
Contributor

ref #58

add option to silence console.log when error occurs while reading and parsing files

@jcblw
Copy link
Collaborator

jcblw commented Mar 31, 2015

I prefer this over #61 👍

@wzrdtales
Copy link

I don't prefer my or this solution. I'm fine with both, it would be great to get the ability to handle the error on the application that uses the module, but that's the only difference. ;)

@maxbeatty
Copy link
Contributor Author

Just thought of how you could control output based on environment in case you want the verboseness during development but not in other environments:

require('dotenv').config({silent: process.env.NODE_ENV !== 'development'})

jcblw added a commit that referenced this pull request Apr 1, 2015
@jcblw jcblw merged commit 72e919f into master Apr 1, 2015
@maxbeatty maxbeatty deleted the silent-errors branch April 13, 2017 03:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants