Skip to content

Unexport plugin commands#6326

Merged
Benehiko merged 1 commit into
docker:masterfrom
Benehiko:command/plugin
Aug 20, 2025
Merged

Unexport plugin commands#6326
Benehiko merged 1 commit into
docker:masterfrom
Benehiko:command/plugin

Conversation

@Benehiko

@Benehiko Benehiko commented Aug 20, 2025

Copy link
Copy Markdown
Member

This patch deprecates exported plugin commands and moves the implementation details to an unexported function.

Commands that are affected include:

  • plugin.NewPluginCommand

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

Go SDK: cli/command/plugin: deprecate `NewPluginCommand`. This function will be removed in the next release.

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

This patch deprecates exported plugin commands and moves the implementation
details to an unexported function.

Commands that are affected include:

- plugin.NewPluginCommand

Signed-off-by: Alano Terblanche <18033717+Benehiko@users.noreply.github.com>
@Benehiko Benehiko added impact/changelog impact/deprecation kind/refactor PR's that refactor, or clean-up code area/go-sdk Changes affecting the Go SDK impact/go-sdk Noteworthy (compatibility changes) in the Go SDK labels Aug 20, 2025
@Benehiko Benehiko added this to the 29.0.0 milestone Aug 20, 2025
@Benehiko Benehiko enabled auto-merge August 20, 2025 12:05
@Benehiko Benehiko requested a review from thaJeztah August 20, 2025 12:05
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

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

Files with missing lines Patch % Lines
cli/command/plugin/cmd.go 0.00% 14 Missing ⚠️
cli/command/commands/commands.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Benehiko Benehiko merged commit d071c29 into docker:master Aug 20, 2025
121 of 127 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/go-sdk Changes affecting the Go SDK impact/changelog impact/deprecation impact/go-sdk Noteworthy (compatibility changes) in the Go SDK kind/refactor PR's that refactor, or clean-up code process/cherry-picked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants