Skip to content

Conversation

@thaJeztah
Copy link
Member

These functions and types are shallow wrappers around the context store and were intended for internal use as implementation for the CLI itself.

They were exported in 3126920 to be used by plugins and Docker Desktop. However, there's currently no public uses of this, and Docker Desktop does not use these functions. These were deprecated in 95eeafa and are no longer used.

This patch removes the deprecated functions as they were meant to be implementation specific for the CLI. If there's a need to provide utilities for manipulating the context-store other than through the CLI itself, we can consider creating an SDK for that purpose.

This removes:

  • RunCreate and CreateOptions
  • RunExport and ExportOptions
  • RunImport
  • RunRemove and RemoveOptions
  • RunUpdate and UpdateOptions
  • RunUse

- How to verify it

- Human readable description for the release notes

Go SDK: cli/command/context: remove deprecated `RunCreate` and `CreateOptions`
Go SDK: cli/command/context: remove deprecated `RunExport` and `ExportOptions`
Go SDK: cli/command/context: remove deprecated `RunImport`
Go SDK: cli/command/context: remove deprecated `RunRemove` and `RemoveOptions`
Go SDK: cli/command/context: remove deprecated `RunUpdate` and `UpdateOptions`
Go SDK: cli/command/context: remove deprecated `RunUse`

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

These functions and types are shallow wrappers around the context
store and were intended for internal use as implementation for the
CLI itself.

They were exported in 3126920 to be
used by plugins and Docker Desktop. However, there's currently no public
uses of this, and Docker Desktop does not use these functions. These were
deprecated in 95eeafa and are no longer
used.

This patch removes the deprecated functions as they were meant to be
implementation specific for the CLI. If there's a need to provide
utilities for manipulating the context-store other than through the
CLI itself, we can consider creating an SDK for that purpose.

This removes:

- `RunCreate` and `CreateOptions`
- `RunExport` and `ExportOptions`
- `RunImport`
- `RunRemove` and `RemoveOptions`
- `RunUpdate` and `UpdateOptions`
- `RunUse`

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 29.0.0 milestone Sep 1, 2025
@thaJeztah thaJeztah added impact/deprecation status/2-code-review area/context 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 Sep 1, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah requested a review from Benehiko September 1, 2025 08:39
@thaJeztah thaJeztah merged commit a2c8862 into docker:master Sep 1, 2025
122 of 124 checks passed
@thaJeztah thaJeztah deleted the rm_exported_context_funcs branch September 1, 2025 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/context area/go-sdk Changes affecting the Go SDK impact/deprecation impact/go-sdk Noteworthy (compatibility changes) in the Go SDK 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