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

[pt-br] feat: add translation of index docs #4868

Merged
merged 9 commits into from
Jul 25, 2024

Conversation

EzzioMoreira
Copy link
Contributor

@EzzioMoreira EzzioMoreira commented Jul 23, 2024

Add a translation of the index /docs to Portuguese.


Preview: https://deploy-preview-4868--opentelemetry.netlify.app/pt/docs/

Copy link

linux-foundation-easycla bot commented Jul 23, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

@EzzioMoreira
Copy link
Contributor Author

Hi @svrnm, can you help me?

Where I find out the default_lang_commit?

Question: I need to add the language pt on line 43 of the package.json file.

@EzzioMoreira
Copy link
Contributor Author

@svrnm I found the command that generate the default_lang_commit npm run fix:i18n:new, disagree the last message rsrsrsrsrsrsrs.

@svrnm
Copy link
Member

svrnm commented Jul 24, 2024

@EzzioMoreira good to hear that you found it!

Copy link

@Vinaum8 Vinaum8 left a comment

Choose a reason for hiding this comment

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

LGTM!

@EzzioMoreira EzzioMoreira marked this pull request as ready for review July 24, 2024 13:05
@EzzioMoreira EzzioMoreira requested a review from a team July 24, 2024 13:05
@cassioliveiram
Copy link

Thanks @EzzioMoreira , looks good to me.

content/pt/docs/_index.md Outdated Show resolved Hide resolved
content/pt/docs/_index.md Outdated Show resolved Hide resolved
content/pt/docs/_index.md Outdated Show resolved Hide resolved
@chalin chalin force-pushed the feat/add-pt-br-index-docs branch 2 times, most recently from cfa3087 to a73992a Compare July 25, 2024 16:20
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.

Let's put this on hold until the following lands:

@chalin chalin force-pushed the feat/add-pt-br-index-docs branch from a73992a to c4443de Compare July 25, 2024 21:06
@chalin chalin requested a review from emdneto July 25, 2024 21:20
@chalin
Copy link
Contributor

chalin commented Jul 25, 2024

@emdneto, @joaopgrassi, @jpkrohling - as nominated docs-pt-approvers, can one of you officially sign off on this PR pls?

@chalin
Copy link
Contributor

chalin commented Jul 25, 2024

All checks are passing except for component-owners, which is a known issue:

1 failing, 15 successful, and 1 neutral checks

Thanks for the formal approval @emdneto. Merging.

@chalin chalin merged commit cfcc8f6 into open-telemetry:main Jul 25, 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
Development

Successfully merging this pull request may close these issues.

8 participants