Skip to content

Add container registry api support#1108

Merged
Toa741 merged 2 commits into
mainfrom
add-container-registry-api-support
Jun 23, 2026
Merged

Add container registry api support#1108
Toa741 merged 2 commits into
mainfrom
add-container-registry-api-support

Conversation

@msrouchou

Copy link
Copy Markdown
Contributor

Adds IContainerRegistryClient with methods to list repositories, list tags, delete a tag, and delete a repository. Includes mock implementation for tests.

Adds IContainerRegistryClient with methods to list repositories, list tags, delete a tag, and delete a repository. Includes mock implementation for tests.
@msrouchou msrouchou marked this pull request as ready for review June 22, 2026 18:08
@msrouchou msrouchou requested a review from a team as a code owner June 22, 2026 18:08
@msrouchou msrouchou requested review from Toa741 and removed request for a team June 22, 2026 18:08
Comment thread NGitLab.Mock/Project.cs

public ProtectedBranchCollection ProtectedBranches { get; }

public ContainerRepositoryCollection ContainerRepositories { get; }

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add the possibility to set it in the GitLabHelpers (to construct GitLabConfig), thus, to be able to use the mocked client.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed ^^

Adds WithContainerRepository on GitLabProject so tests can configure mock container registries via the config API.
@msrouchou msrouchou force-pushed the add-container-registry-api-support branch from 610cf77 to 1f5ca62 Compare June 23, 2026 15:37
@Toa741 Toa741 merged commit 6b995ab into main Jun 23, 2026
7 checks passed
@Toa741 Toa741 deleted the add-container-registry-api-support branch June 23, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants