-
Notifications
You must be signed in to change notification settings - Fork 11
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
command-line params don't override config #65
Comments
❯ pip show milc
Name: milc
Version: 1.6.5
Summary: Opinionated Batteries-Included Python 3 CLI Framework.
Home-page: https://milc.clueboard.co/
Author: skullydazed
Author-email: [email protected]
License: MIT License
Location: /home/kbingham/.pyenv/versions/3.9.0/lib/python3.9/site-packages
Requires: appdirs, argcomplete, colorama, halo, spinners
Required-by: netfoundry
|
Thanks for the report! I've fixed it in c901c3b, can you give that a try? If that works for you I'll cut a new release. |
Wow! Thank you so much. That was quick, and it's encouraging to see the batteries are still included with MILC. This is the first CLI framework I've worked with and it's been a pleasure. Confirmed. With that rev I am able to override the general config with either space or equal sep'd command-line params. |
Glad to hear it! I've released 1.6.6 with that fix. |
This works correctly:
# overrides config $ mycli --myopt myarg
This does not override
# config overrides command-line $ mycli --myopt=myarg
The only difference seems to be whether space-sep or
=
-sep'd opt=arg syntax is used. I'm investigating and will update this issue if I find the cause.The text was updated successfully, but these errors were encountered: