Skip to content

internal/registryclient: minor cleanups - #6744

Merged
vvoland merged 2 commits into
docker:masterfrom
thaJeztah:registryclient_cleanups
Jan 16, 2026
Merged

internal/registryclient: minor cleanups#6744
vvoland merged 2 commits into
docker:masterfrom
thaJeztah:registryclient_cleanups

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

- Human readable description for the release notes

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

@thaJeztahthaJeztah added this to the 29.2.0 milestone Jan 14, 2026
@thaJeztahthaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Jan 14, 2026
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
- remove constructor
- fix mixed pointer/non-pointer receivers
- just embed the error we want to produce
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
thaJeztahforce-pushed the registryclient_cleanups branch from f2a32c5 to 816f455CompareJanuary 14, 2026 23:06
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

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

Files with missing linesPatch %Lines
internal/registryclient/fetcher.go0.00%4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@vvoland
vvoland merged commit 714f5bf into docker:masterJan 16, 2026
91 checks passed
@thaJeztah
thaJeztah deleted the registryclient_cleanups branch January 16, 2026 12:17
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactorPR's that refactor, or clean-up codestatus/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@thaJeztah@codecov-commenter@vvoland