Skip to content

Replace "Google Cloud Registry" with "Artifact Registry"#7118

Merged
sarah11918 merged 2 commits into
withastro:mainfrom
shuuji3:update-google-cloud
Mar 1, 2024
Merged

Replace "Google Cloud Registry" with "Artifact Registry"#7118
sarah11918 merged 2 commits into
withastro:mainfrom
shuuji3:update-google-cloud

Conversation

@shuuji3
Copy link
Copy Markdown
Contributor

@shuuji3 shuuji3 commented Mar 1, 2024

Description (required)

A while ago, Goole Cloud deprecated Container Registry in favor of Artifact Registry so the linked documentation page is also updated. Artifact Registry is basically a superset of Container Registry so the usage and documentation content is the same.

This PR replaces the service name and its links.

🌐 Preview: https://docs-git-fork-shuuji3-update-google-cloud-astrodotbuild.vercel.app/en/guides/deploy/google-cloud/

Related issues & labels (optional)

  • Closes #
  • Suggested label:

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview Mar 1, 2024 1:52pm


**Build locally using Docker**:

Use the `docker build` command to build the image, `docker tag` to give it a tag, then `docker push` to push it to a registry. In the case of Google Cloud, [`Google Cloud Registry`](https://cloud.google.com/container-registry/docs/pushing-and-pulling#add-registry) is the easiest option, but you can also use [Docker Hub](https://hub.docker.com/).
Copy link
Copy Markdown
Contributor Author

@shuuji3 shuuji3 Mar 1, 2024

Choose a reason for hiding this comment

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

The original link directly points to #add-registry but it was skipping a few prerequisites before that section. For example, enabling API is a necessary operation for every new Google Cloud Project so I think it should navigate to the beginning of the page.

Copy link
Copy Markdown
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you for being on top of this! 💪

@sarah11918 sarah11918 added typo/link/grammar - quick fix! Quick fix for a link, typo, grammar/syntax error etc. Doesn't significantly change meaning of text. Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! labels Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! typo/link/grammar - quick fix! Quick fix for a link, typo, grammar/syntax error etc. Doesn't significantly change meaning of text.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants