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

Disable ".env not found" warning by default #57

Merged
merged 3 commits into from
Aug 23, 2017

Conversation

maxkoryukov
Copy link
Contributor

@maxkoryukov maxkoryukov commented May 26, 2017

Because this warning scares me and our team more often than helps us.

Probably, similar changes should be applied to other functions.

Added

verbose option for load_dotenv, which will log any error messages

Changed

Warning File .env not found disabled by default.

Several investigations are here: motdotla/dotenv#154 (be careful, NodeJS thread)

@coveralls
Copy link

coveralls commented May 26, 2017

Coverage Status

Coverage increased (+0.1%) to 82.514% when pulling 311f56c on maxkoryukov:fix/disable-warn-on-not-exist into e5d68ff on theskumar:master.

@ghost
Copy link

ghost commented Aug 21, 2017

This is very nice, I want the feature too. 👍

@maxkoryukov
Copy link
Contributor Author

ping @theskumar

@theskumar theskumar merged commit 7ab249b into theskumar:master Aug 23, 2017
theskumar pushed a commit that referenced this pull request Aug 23, 2017
@theskumar
Copy link
Owner

Thank you ! :)

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.

None yet

3 participants