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

Upgrade weather-dl config language to allow updates from continuous integration services. #267

Open
alxmrs opened this issue Dec 8, 2022 · 0 comments

Comments

@alxmrs
Copy link
Collaborator

alxmrs commented Dec 8, 2022

Right now, weather-dl and it's configs are oriented towards manual invocation. It would be quite useful to be able to run this tool in a CI, where CLI arguments would stay the same and the config would change.

Implementation notes

A simple way to implement this could be to just upgrade our ConfigParser class to one that supports interpolation. It looks like the majority of the capabilities we want already exist with the default parser, however, it would be best to make sure this doesn't interfere with other language features.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant