Skip to content

cmd/dockerd: remove various cobra templating#46871

Merged
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:less_cobra_customisation
Dec 1, 2023
Merged

cmd/dockerd: remove various cobra templating#46871
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:less_cobra_customisation

Conversation

@thaJeztah
Copy link
Member

cmd/dockerd: remove various cobra templating

This removes various templating functions that were added for the
docker CLI. These are not needed for the dockerd binary, which does
not have subcommands or management commands.

Revert "Only hide commands if the env variable is set."

This reverts commit a7c8bca.

cmd/dockerd: make cobra templates a const

- How to verify it

- Description for the changelog

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

@thaJeztah thaJeztah added area/cli Client status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Nov 30, 2023
This removes various templating functions that were added for the
docker CLI. These are not needed for the dockerd binary, which does
not have subcommands or management commands.

Revert "Only hide commands if the env variable is set."

This reverts commit a7c8bca.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the less_cobra_customisation branch from a7b02e9 to 61d9485 Compare November 30, 2023 12:40
@thaJeztah thaJeztah changed the title Less cobra customisation cmd/dockerd: remove various cobra templating Nov 30, 2023
@thaJeztah thaJeztah self-assigned this Nov 30, 2023
@thaJeztah thaJeztah merged commit bb99b3f into moby:master Dec 1, 2023
@thaJeztah thaJeztah deleted the less_cobra_customisation branch December 1, 2023 09:58
@thaJeztah thaJeztah added this to the 25.0.0 milestone Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli Client kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants