Skip to content

Added "Converting types" #293

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

Closed
wants to merge 3 commits into from
Closed

Added "Converting types" #293

wants to merge 3 commits into from

Conversation

guettli
Copy link

@guettli guettli commented Dec 16, 2020

No description provided.

@coveralls
Copy link

coveralls commented Dec 16, 2020

Coverage Status

Coverage remained the same at 88.309% when pulling 2144ee7 on guettli:patch-1 into 26ff5b7 on theskumar:master.

@sergeyklay
Copy link
Contributor

@bbc2 Could you please take a look? Can we have this in the docs?

@bbc2
Copy link
Collaborator

bbc2 commented Feb 21, 2021

Thank you for the suggestion but I don't think it's worth having this in the readme because the fact that os.getenv returns a string is already documented: https://docs.python.org/3.9/library/os.html#os.getenv. Python-dotenv doesn't change that.

@ramwin
Copy link

ramwin commented Apr 25, 2023

I like this commit. If this was merge into main branch, It really saves newbie a lot of time. And since it only add some suggestions to the README.md file, It didn't break the package's behaviour.

@alexeyshockov
Copy link

Yeah, such examples would be definitely appreciated. IMO there are no reasons to not mention some existing env vars technics.

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.

6 participants