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

Default option group: make it globally configurable #3

Closed
janluke opened this issue Jan 19, 2021 · 2 comments
Closed

Default option group: make it globally configurable #3

janluke opened this issue Jan 19, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@janluke
Copy link
Owner

janluke commented Jan 19, 2021

This is a minor enhancement, a refinement, to implement (maybe) in the future.

Specs

  • The name and the help of the default OptionGroup should be globally configurable.
  • Important: as it is implemented now, if the command has no option groups defined, the default group name should be just "Options" and the help should be empty.

Notes

  • There should be an analogous issue relative to subcommand sections; these configurations should be released together. Subcommand sections don't strictly need this, because no command is added automatically by Cloup (as it happens in the case of options with --help), so if the user wants to change "Other commands" to something else, (s)he can define another a section and leave the default one empty.
@janluke janluke added enhancement New feature or request good first issue Good for newcomers labels Jan 19, 2021
@janluke janluke removed the good first issue Good for newcomers label Feb 5, 2021
@janluke janluke changed the title Default option group: make the name and help globally configurable Default option group: make it globally configurable Apr 21, 2021
@janluke
Copy link
Owner Author

janluke commented Apr 21, 2021

Related: #15. This feature can be implemented with a context attribute

default_option_group: OptionGroup = OptionGroup('Other options')

@janluke
Copy link
Owner Author

janluke commented Aug 5, 2021

I don't feel this deserves yet another setting. If one really wants to change the default, one can already override the get_default_option_group method.

@janluke janluke closed this as completed Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant