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

i18n(zh-cn): update guides/deploy/github.mdx #5156

Merged
merged 3 commits into from
Oct 26, 2023

Conversation

100gle
Copy link
Contributor

@100gle 100gle commented Oct 20, 2023

Description (required)

update guides/deploy/github.mdx

Related issues & labels (optional)

#5131

@netlify
Copy link

netlify bot commented Oct 20, 2023

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 6d32f66
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/653a875634a50300085106e8
😎 Deploy Preview https://deploy-preview-5156--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Oct 20, 2023
@100gle
Copy link
Contributor Author

100gle commented Oct 20, 2023

@Yan-Thomas the failed reason of checking CI is so weired. Even I didn't change anything about sitemap or other languages version. How to solve it?

@yanthomasdev
Copy link
Member

@Yan-Thomas the failed reason of checking CI is so weired. Even I didn't change anything about sitemap or other languages version. How to solve it?

Oh, this happens because sometimes GitHub fails to send the data we need for the contributors/translators list, so this issue happens. Rerunning the CI (be it manually or by adding a commit) solves it.

Copy link
Member

@liruifengv liruifengv left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks peeps 🚀

@yanthomasdev yanthomasdev merged commit eb2067f into withastro:main Oct 26, 2023
9 checks passed
@100gle 100gle deleted the github-patch-20231020 branch October 28, 2023 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants