Skip to content

Conversation

@thaJeztah
Copy link
Member

The allManagementSubCommands function is used to present plugin-commands in the docker --help output; these commands are included in the "management commands" section, but for plugins we don't know if they have sub-commands.

However, plugin stubs may be hidden (for placeholders that are not yet loaded), or not be runnable, which was previously ignored.

This patch treats plugin-stubs the same as other commands, with the exception of checking if they have subcommands (which is not yet known for plugin-stubs).

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

The allManagementSubCommands function is used to present plugin-commands
in the docker --help output; these commands are included in the "management
commands" section, but for plugins we don't know if they have sub-commands.

However, plugin stubs may be hidden (for placeholders that are not yet loaded),
or not be runnable, which was previously ignored.

This patch treats plugin-stubs the same as other commands, with the exception
of checking if they have subcommands (which is not yet known for plugin-stubs).

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/cobra.go 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah marked this pull request as ready for review November 3, 2025 13:28
@thaJeztah thaJeztah requested a review from Benehiko November 3, 2025 13:29
@thaJeztah thaJeztah merged commit 7e32fdf into docker:master Nov 3, 2025
112 of 113 checks passed
@thaJeztah thaJeztah deleted the skip_hidden_plugin_commands branch November 3, 2025 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants