Skip to content

[28.x backport] cli/command: fix godoc links and inline resolveAuthConfigFromImage#6315

Merged
thaJeztah merged 2 commits intodocker:28.xfrom
thaJeztah:28.x_backport_auth_cleanups
Aug 20, 2025
Merged

[28.x backport] cli/command: fix godoc links and inline resolveAuthConfigFromImage#6315
thaJeztah merged 2 commits intodocker:28.xfrom
thaJeztah:28.x_backport_auth_cleanups

Conversation

@thaJeztah
Copy link
Member


cli/command: fix godoc links

  • Use versioned links to github.com/docker/docker packages
  • Fix links to RFC 4648, section 5

cli/command: inline resolveAuthConfigFromImage

@thaJeztah thaJeztah added this to the 28.4.0 milestone Aug 19, 2025
@thaJeztah thaJeztah added status/2-code-review kind/bugfix PR's that fix bugs kind/refactor PR's that refactor, or clean-up code labels Aug 19, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
cli/command/registry.go 83.33% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

- Use versioned links to github.com/docker/docker packages
- Fix links to RFC 4648, section 5

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 2d3b0b3)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 4286883)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah force-pushed the 28.x_backport_auth_cleanups branch from dcc2262 to 942eade Compare August 19, 2025 21:42
@thaJeztah thaJeztah requested a review from Benehiko August 20, 2025 07:05
@thaJeztah thaJeztah merged commit 5a80c3e into docker:28.x Aug 20, 2025
100 of 101 checks passed
@thaJeztah thaJeztah deleted the 28.x_backport_auth_cleanups branch August 20, 2025 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bugfix PR's that fix bugs 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