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

[zh] add contributing/style-guide.md #4685

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

darielli
Copy link
Contributor

[zh] add contributing/style-guide.md

@darielli darielli requested review from a team June 14, 2024 11:56
Copy link
Member

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Interesting! I wonder if the style guide makes sense for a tech writer writing in Chinese.

@reyang
Copy link
Member

reyang commented Jun 14, 2024

Interesting! I wonder if the style guide makes sense for a tech writer writing in Chinese.

I just checked and can confirm that that style guide makes sense for Chinese.

@chalin chalin force-pushed the style-guide branch 2 times, most recently from 03f4cc0 to 5ffbc35 Compare June 14, 2024 22:33
Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

This page will need to be updated to conform to https://opentelemetry.io/docs/contributing/localization/#translation-tips, in particular header IDs are missing.

@chalin chalin requested a review from a team July 24, 2024 22:18
Copy link
Contributor

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Ok, let's get this merged as is. I'll let @open-telemetry/docs-zh-approvers address the required updates (that I mentioned in my previous comment) to the page.

@chalin
Copy link
Contributor

chalin commented Jul 24, 2024

Ignoring Component-owners GH action failure for now and merging, since all other checks are green:

image

@chalin chalin merged commit 82c7249 into open-telemetry:main Jul 24, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants