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.

Sign up for GitHub

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

Jump to bottom

Use a separate config file for settings #11

Closed
surak opened this issue Oct 10, 2018 · 2 comments
Closed

Use a separate config file for settings #11

surak opened this issue Oct 10, 2018 · 2 comments
Labels
fixed! If something's fixed, it should have this label improvement This is an improvement outdated This issue is from a previous release and is most likey not relevant. (Archived)

Comments

@surak
Copy link
Contributor

surak commented Oct 10, 2018

This way, I won't push my key by mistake here :-)
surak added a commit to surak/E-Paper-Calendar-with-iCal-sync-and-live-weather that referenced this issue Oct 10, 2018
@surak
closing issues aceinnolab#9 and aceinnolab#11
ff1ec85
@surak surak mentioned this issue Oct 10, 2018
Merged
aceisace added a commit that referenced this issue Oct 11, 2018
@aceisace
Merge pull request #12 from surak/master
24eacca 
closing issues #9 and #11
@aceisace
Copy link
Member

I've merged your pull request.
Yes, it's a good idea to have a seperate configuration file.

Thanks for contributing and improving the script :)

@aceisace
Copy link
Member

I'm closing this issue for now. If you think that this issue should remain open, please let me know.

@aceisace aceisace added improvement This is an improvement fixed! If something's fixed, it should have this label labels Oct 22, 2018
@aceisace aceisace added the outdated This issue is from a previous release and is most likey not relevant. (Archived) label Apr 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fixed! If something's fixed, it should have this label improvement This is an improvement outdated This issue is from a previous release and is most likey not relevant. (Archived)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@surak @aceisace