Skip to content

internal/registry: remove RepositoryInfo, add NewIndexInfo - #6234

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:ParseRepositoryInfo_no_err_step2
Aug 4, 2025
Merged

internal/registry: remove RepositoryInfo, add NewIndexInfo#6234
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:ParseRepositoryInfo_no_err_step2

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

Most places only use IndexInfo (and may not even need that), so replace the use of ParseRepositoryInfo for NewIndexInfo, and move the RepositoryInfo type to the trust package, which uses it as part of its ImageRefAndAuth struct.

- Human readable description for the release notes

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

Most places only use IndexInfo (and may not even need that), so replace
the use of ParseRepositoryInfo for NewIndexInfo, and move the RepositoryInfo
type to the trust package, which uses it as part of its ImageRefAndAuth
struct.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
MemberAuthor

@Benehiko ptal - it's not the final state, but a stepping stone towards removing more of these extra layers (after this, I need to look at some of the "content trust" bits to remove their special types).

@thaJeztah
thaJeztah merged commit fcfaa8d into docker:masterAug 4, 2025
102 checks passed
@thaJeztah
thaJeztah deleted the ParseRepositoryInfo_no_err_step2 branch August 4, 2025 13:21
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@thaJeztah@Benehiko