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

Allow multiple -a args for Stetl main prog #72

Closed
justb4 opened this issue Apr 27, 2018 · 1 comment
Closed

Allow multiple -a args for Stetl main prog #72

justb4 opened this issue Apr 27, 2018 · 1 comment
Assignees
Milestone

Comments

@justb4
Copy link
Member

justb4 commented Apr 27, 2018

Currently only one -a argument can be passed to set either a list of options or a single options (.args) file. Allowing multiple -a arguments allows for more simpler overriding of for example default options. For example:

stetl -c my.cfg -a default.args -a my.args

or

stetl -c my.cfg -a default.args -a db_host=host -a db_user=me -a db_password=xyz

This allows to keep all default args and in my.args or explicit settings just have a few options like passwords.
The order of the -a args will determine overriding order. Args passed via the Environment like stetl_db_password will still prevail over any commandline args.

@justb4
Copy link
Member Author

justb4 commented May 3, 2018

This has been fixed and can be closed. Supports the following issue:
nlextract/NLExtract#249 and Docker deployments within SE:
https://github.com/smartemission/smartemission

@justb4 justb4 closed this as completed May 3, 2018
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