Refactor DockerRegistry hierarchy and dependencies

dotnet / docker-tools

This is a repo to house some common tools for our various docker repos.

MIT License

124 stars 46 forks source link

Closed NikolaMilosavljevic closed 5 months ago

NikolaMilosavljevic commented 5 months ago

Removes DockerRegistryCommand from hierarchy, so BuildCommand and PublishManifestCommand inherit from ManifestCommand.
Gets rid of DockerRegistryOptions altogether as it contained just RegistryCredentialsOptions which is independently composable.
Moves ExecuteWithCredentialsAsync method to a new file for RegistryCredentialsProvider extensions.

dotnet-issue-labeler[bot] commented 5 months ago

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

dotnet-issue-labeler[bot] commented 5 months ago

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.