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

#26: Adding an "overwrite" flag to read_dotenv, False by default. #27

Merged
merged 4 commits into from Dec 11, 2017

Conversation

alexdlaird
Copy link
Contributor

PR for issue #26.

Allows for an optional "overwrite" parameter, False by default (so nothing existing breaks). Documented as well.

This would especially be useful when running tests, so test cases can still rely on a .env file and ensure it will override pre-existing environment variables that may not be suitable to use when running tests.

@alexdlaird
Copy link
Contributor Author

@jpadilla All tests passing. What's your release schedule look like for this? We would love to be able to use this in our project, but this issue is causing breakage in our tests—wondering if we can rely on this soon being available on PyPi or if we should make our own dependency?

@jpadilla jpadilla merged commit 4a3ea82 into jpadilla:master Dec 11, 2017
@jpadilla
Copy link
Owner

@alexdlaird thanks you for working on this! I'll publish a new version today.

@jpadilla
Copy link
Owner

@lightstrike
Copy link

Really helpful, thanks @alexdlaird and @jpadilla!

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