-
Notifications
You must be signed in to change notification settings - Fork 349
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
feat: dot notation #789
feat: dot notation #789
Conversation
Still likely need to add more tests and some documentation updates |
from discussion on gitter |
Codecov ReportBase: 99.29% // Head: 99.30% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #789 +/- ##
=======================================
Coverage 99.29% 99.30%
=======================================
Files 16 16
Lines 3974 4022 +48
=======================================
+ Hits 3946 3994 +48
Misses 28 28
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
15a8614
to
2693b60
Compare
@henryiii I think this is ready now |
Is there a test for toml config with this? That was one of the reasons for it IIRC? |
The toml processing of dot notation worked before and is tested in a few locations, so this just adds to the command line parsing and doesn't change anything in the file parsing. |
Add the ability to use subcommand options using . notation
--sub.option
or nested subcommands using dot notation.