Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support artifact registry #207

Merged
merged 3 commits into from
Nov 19, 2023

Conversation

benjaminch
Copy link
Contributor

@benjaminch benjaminch commented Nov 13, 2023

First draft version adding support for Google Artifact registry.

Can you let me know what you think about it? I focused on registry + docker images in the examples.

Issue: #208

@benjaminch benjaminch force-pushed the feat/support-artifact-registry branch from ce81631 to 0bb663a Compare November 13, 2023 23:20
artifact-registry/src/client.rs Outdated Show resolved Hide resolved
artifact-registry/src/client.rs Show resolved Hide resolved
artifact-registry/src/sign.rs Outdated Show resolved Hide resolved
@benjaminch
Copy link
Contributor Author

Thanks for review @yoshidan, pushed a patch with fixes :)

@benjaminch benjaminch force-pushed the feat/support-artifact-registry branch from 795dee5 to 4eb9cf5 Compare November 16, 2023 09:20
@benjaminch benjaminch force-pushed the feat/support-artifact-registry branch from 4eb9cf5 to 884281d Compare November 16, 2023 13:27
@yoshidan yoshidan added the safe to test safe to test label Nov 19, 2023
@benjaminch
Copy link
Contributor Author

Hi @yoshidan ! Should I add Artifact registry to the CI? Totally forgot.

.github/workflows/ci.yaml

@yoshidan yoshidan merged commit 58ee24d into yoshidan:main Nov 19, 2023
5 checks passed
@yoshidan
Copy link
Owner

I will add the CI for artifact registry.
Thanks!

@benjaminch
Copy link
Contributor Author

Thanks a lot 😉

@benjaminch
Copy link
Contributor Author

Would you mind creating a new release with this change?
Thanks a lot :)

@yoshidan
Copy link
Owner

I will publish crate after adding CI. Please wait a few days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test safe to test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants