Skip to content

[pt] feat: add doc br pull request content/pt/docs/contributing/pull-requests.md #7094

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

Merged

Conversation

EzzioMoreira
Copy link
Contributor

@EzzioMoreira EzzioMoreira commented Jun 12, 2025

This pull request includes a change to the content/pt/docs/contributing/pull-requests.md file to improve the readability and accuracy of the documentation. The most important change involves adding pull request documentation.

Closes: #6853

Copy link
Contributor

@maryliag maryliag left a comment

Choose a reason for hiding this comment

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

🚀

@maryliag maryliag added the docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer label Jun 12, 2025
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.

Thanks! Some headings are missing IDs, please add them.

@chalin chalin removed the docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer label Jun 15, 2025
@chalin chalin force-pushed the feat/localization-pr-contrubutor branch from ae04f2b to b6937f1 Compare June 15, 2025 17:50
@opentelemetrybot opentelemetrybot requested review from a team June 15, 2025 17:50
Copy link
Member

@vitorvasc vitorvasc left a comment

Choose a reason for hiding this comment

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

Muito bom 🚀

Deixei como sugestão os anchors que estão faltando nos headings!

Copy link
Member

@emdneto emdneto left a comment

Choose a reason for hiding this comment

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

LGTM with the fixes

@opentelemetrybot opentelemetrybot requested a review from a team June 24, 2025 23:06
@EzzioMoreira
Copy link
Contributor Author

/fix:all

@opentelemetrybot
Copy link
Collaborator

EzzioMoreira and others added 2 commits June 25, 2025 06:49
Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
@vitorvasc vitorvasc force-pushed the feat/localization-pr-contrubutor branch from c08c5a7 to 67e19dd Compare June 25, 2025 09:49
Copy link
Member

@vitorvasc vitorvasc left a comment

Choose a reason for hiding this comment

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

🥳 🚀

@vitorvasc vitorvasc added the docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer label Jun 25, 2025
@opentelemetrybot opentelemetrybot requested a review from a team June 25, 2025 14:27
@theletterf theletterf added this pull request to the merge queue Jun 27, 2025
Merged via the queue into open-telemetry:main with commit f825cb1 Jun 27, 2025
19 checks passed
svrnm pushed a commit to svrnm/opentelemetry.io that referenced this pull request Jun 27, 2025
…requests.md (open-telemetry#7094)

Co-authored-by: Vitor Vasconcellos <vvasconcellos1@gmail.com>
Co-authored-by: Marylia Gutierrez <marylia.gutierrez@grafana.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer lang:pt
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[pt] Localize docs/contributing/pull-requests.md
7 participants