-
Notifications
You must be signed in to change notification settings - Fork 0
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
Load multiple .env files with load_dotenv
#5
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/br3ndonland/fastenv/2EjMhUXmC9wSi6dw6o3Gg6tdS58k |
Codecov Report
@@ Coverage Diff @@
## develop #5 +/- ##
========================================
Coverage 100.0% 100.0%
========================================
Files 3 3
Lines 104 113 +9
========================================
+ Hits 104 113 +9
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Based on @sourcery-ai bot feedback on #5 #5 (comment)
Based on @sourcery-ai bot feedback on #5
Based on @sourcery-ai bot feedback on #5
78036c1
to
f9ef9a7
Compare
Based on @sourcery-ai bot feedback on #5
Based on @sourcery-ai bot feedback on #5
f9ef9a7
to
74ca273
Compare
Sourcery Code Quality Report✅ Merging this PR will increase code quality in the affected files by 3.19%.
Here are some functions in these files that still need a tune-up:
Legend and ExplanationThe emojis denote the absolute quality of the code:
The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request. Please see our documentation here for details on how these metrics are calculated. We are actively working on this report - lots more documentation and extra metrics to come! Help us improve this quality report! |
#5 The `load_dotenv` method originally had a default filename argument of `".env"`. This default was removed in #5, when the ability to load multiple .env files was implemented. The default filename argument could potentially be added back in the future, but adding the same default to the examples in the docs will help ensure the examples work, regardless of whether or not `load_dotenv` supplies a default filename argument.
Description
This PR will update
load_dotenv
to load multiple .env files in a single call.Changes
load_dotenv
(0813a5e)from __future__ import annotations
(83b7696)load_dotenv
into smaller methods (74ca273)Related
#2