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

renaming image-copy-gcr to image-copy-gcp #206

Merged
merged 1 commit into from
May 25, 2024
Merged

renaming image-copy-gcr to image-copy-gcp #206

merged 1 commit into from
May 25, 2024

Conversation

SharpRake
Copy link
Contributor

I thought it made sense to update this example (which is useful and works perfectly fine) to reflect recent changes to GCP's product offerings. They recently deprecated GCR and the service will be totally shut down in 2025.

This PR updates the name of this example to image-copy-gcp and updates other docs in this repository to reflect the change.

After updating it ✨ works on my machine ✨ but I may very well have missed something important, so please test things out yourself.

Signed-off-by: Mark Drake <mark@chainguard.dev>
@SharpRake SharpRake requested a review from imjasonh May 24, 2024 21:25
Copy link
Member

@imjasonh imjasonh left a comment

Choose a reason for hiding this comment

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

Do we have links to this directory in edu docs? I wish we could set up a redirect on GitHub's side :(

In any case, LGTM, and thanks for doing this!

@imjasonh imjasonh merged commit 6320b49 into main May 25, 2024
11 checks passed
@imjasonh imjasonh deleted the gcr2gcp branch May 25, 2024 00:15
@SharpRake
Copy link
Contributor Author

Do we have links to this directory in edu docs? I wish we could set up a redirect on GitHub's side :(

Just double checked - nope! I'm working on the companion doc for this example now and we haven't mentioned on academy yet otherwise.

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.

None yet

2 participants