Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Aug 23, 2025

This function was exported to share it between "trust" and "image", but was only a shallow wrapper, so split the implementations where used.

(cherry picked from commit 7ad113c)

- Human readable description for the release notes

Go SDK: cli/command/image: deprecate `AuthResolver` utility

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

This function was exported to share it between "trust" and "image",
but was only a shallow wrapper, so split the implementations where
used.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
(cherry picked from commit 7ad113c)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 28.4.0 milestone Aug 23, 2025
@thaJeztah thaJeztah added impact/deprecation status/2-code-review 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 area/trust area/authentication labels Aug 23, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 75.00000% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/image/trust.go 33.33% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@thaJeztah thaJeztah requested a review from Benehiko August 25, 2025 12:34
@thaJeztah thaJeztah merged commit 23518a4 into docker:28.x Aug 25, 2025
138 of 141 checks passed
@thaJeztah thaJeztah deleted the 28.x_backport_unexport_authresolver_util branch August 25, 2025 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/authentication area/go-sdk Changes affecting the Go SDK area/trust 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