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

Added support for .env files #131

Merged
merged 3 commits into from
Oct 3, 2023
Merged

Conversation

jrast
Copy link
Contributor

@jrast jrast commented Sep 28, 2023

Added support to load configuration from a .env file.

Let me know if this is of interest. If so I will also add some lines in the readme.

@longstone
Copy link
Collaborator

It's not breaking anything, so from my opinion its fine.
Out of couriosity, you are using withings-sync as cli tool?

setup.py Outdated Show resolved Hide resolved
@jrast
Copy link
Contributor Author

jrast commented Sep 29, 2023

It's not breaking anything, so from my opinion its fine. Out of couriosity, you are using withings-sync as cli tool?

I was using a source checkout with some local modifications for a long time. Now it stopped working and I checked this repo to see if there are updates available. Seems like my local modifications are no longer required with the latest version, so I will use the CLI tool in the future.

@longstone
Copy link
Collaborator

If you can add a section in readme, i'll merge it and create a release. thank you very much!

@longstone longstone merged commit ca6b17c into jaroslawhartman:master Oct 3, 2023
@longstone
Copy link
Collaborator

Thx!

@jrast jrast deleted the dotenv-support branch October 3, 2023 11:56
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.

2 participants