Rework of the subgroups feature + Minor Refactoring

lebrice / SimpleParsing

Simple, Elegant, Typed Argument Parsing with argparse

MIT License

386 stars 47 forks source link

Closed lebrice closed 1 year ago

lebrice commented 1 year ago

WIP.

[x] Add a whole bunch of tests for the subgroups feature (with nesting)
[x] Make nested subgroups work
[x] Make other tests work
[ ] ~~Identify and mark out candidates for subsequent refactoring PRs.~~ (out of scope, this is already a pretty large PR as-is).

lebrice commented 1 year ago

Potential future improvements:

Show all possible subgroups in the "--help" action, not just the selected subgroups.
Make the subgroups function more flexible:
- Allow passing partials or lambdas as default factories or subgroup options