Skip to content

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Sep 2, 2025

relates to:

cli-plugins/manager: remove deprecated IsNotFound

These errors satisfy errdefs.IsNotFound, which can be used instead. This
function was deprecated in 7146021 and
is no longer used.

cli-plugins/manager: remove deprecated metadata aliases

These consts and types were moved to a separate metadata package in commits
292713c and 4321293,
and deprecated in 72f76f2, 5876b29,
and 6fa7d18.

This removes the deprecated aliases in cli-plugins/manager in favor of
their equivalent in cli-plugins/manager/metadata:

  • CommandAnnotationPlugin
  • CommandAnnotationPluginVendor
  • CommandAnnotationPluginVersion
  • CommandAnnotationPluginInvalid
  • CommandAnnotationPluginCommandPath
  • NamePrefix
  • MetadataSubcommandName
  • HookSubcommandName
  • Metadata
  • ReexecEnvvar

- Human readable description for the release notes

Go SDK: Remove deprecated `IsNotFound`, `CommandAnnotationPlugin`, `CommandAnnotationPluginVendor`, `CommandAnnotationPluginVersion`, `CommandAnnotationPluginInvalid`, `CommandAnnotationPluginCommandPath`, `NamePrefix`, `MetadataSubcommandName`, `HookSubcommandName`, `Metadata`, and `ReexecEnvvar` from `cli-plugins/manager` in favor of their `cli-plugins/manager/metadata` equivalents.

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

These errors satisfy errdefs.IsNotFound, which can be used instead. This
function was deprecated in 7146021 and
is no longer used.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
These consts and types were moved to a separate metadata package in commits
292713c and 4321293,
and deprecated in 72f76f2, 5876b29,
and 6fa7d18.

This removes the deprecated aliases in `cli-plugins/manager` in favor of
their equivalent in `cli-plugins/manager/metadata`:

- `CommandAnnotationPlugin`
- `CommandAnnotationPluginVendor`
- `CommandAnnotationPluginVersion`
- `CommandAnnotationPluginInvalid`
- `CommandAnnotationPluginCommandPath`
- `NamePrefix`
- `MetadataSubcommandName`
- `HookSubcommandName`
- `Metadata`
- `ReexecEnvvar`

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 29.0.0 milestone Sep 2, 2025
@thaJeztah thaJeztah added impact/deprecation status/2-code-review area/plugins 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 2, 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 merged commit 153bd95 into docker:master Sep 2, 2025
108 checks passed
@thaJeztah thaJeztah deleted the plugin_rm_deprecated branch September 2, 2025 10:17
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 area/plugins 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