Unexport context command by Benehiko · Pull Request #6322 · docker/cli · GitHub
Skip to content

Unexport context command - #6322

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

Benehiko merged 1 commit into
docker:masterfrom
Benehiko:command/context

Conversation

@Benehiko

Copy link
Copy Markdown
Member

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

Commands that are affected include:

  • context.NewContextCommand

- What I did

- How I did it

- How to verify it

- Human readable description for the release notes

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

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

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

Commands that are affected include:

- context.NewContextCommand

Signed-off-by: Alano Terblanche <18033717+Benehiko@users.noreply.github.com>
@Benehiko Benehiko added this to the 29.0.0 milestone Aug 20, 2025
@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 requested a review from thaJeztah August 20, 2025 11:24
@Benehiko
Benehiko enabled auto-merge August 20, 2025 11:24

@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

@codecov-commenter

Copy link
Copy Markdown

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