-
Notifications
You must be signed in to change notification settings - Fork 420
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
Subcommand with inherited mixed-in standard help options + HelpCommand class: failure #1319
Labels
Milestone
Comments
Good catch! I thought I had fixed this with #1316, but it turns out that the I may need to add special logic for |
I finally had time to look at this. I resolved this by not inheriting the standard help options mixin if a command already has one of the |
MarkoMackic
pushed a commit
to MarkoMackic/picocli
that referenced
this issue
Oct 17, 2021
MarkoMackic
pushed a commit
to MarkoMackic/picocli
that referenced
this issue
Oct 17, 2021
…n parent has inherited mixed-in help options and the built-in `HelpCommand` subcommand. Closes remkop#1319
MarkoMackic
added a commit
to MarkoMackic/picocli
that referenced
this issue
Oct 17, 2021
…ion` when parent has inherited mixed-in help options and the built-in `HelpCommand` subcommand." This reverts commit 1a95bf2.
MarkoMackic
added a commit
to MarkoMackic/picocli
that referenced
this issue
Oct 17, 2021
This reverts commit 88cfd06.
MarkoMackic
added a commit
to MarkoMackic/picocli
that referenced
this issue
Oct 17, 2021
…ion` when parent has inherited mixed-in help options and the built-in `HelpCommand` subcommand." This reverts commit 1a95bf2.
MarkoMackic
added a commit
to MarkoMackic/picocli
that referenced
this issue
Oct 17, 2021
This reverts commit 88cfd06.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Minimum working example:
When running this code, I'm getting
It's the combination of inherited
scope
+mixinStandardHelpOptions
+subcommands = { HelpCommand.class }
that's causing trouble here. If I setscope
toScopeType.LOCAL
, the sample runs fine.The text was updated successfully, but these errors were encountered: