-
Notifications
You must be signed in to change notification settings - Fork 1.4k
[pt] Localize docs/contributing/sig-practices.md #7051
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eu acho que devemos usar "o/do PR" e "a/da branch"
@open-telemetry/docs-pt-approvers thoughts?
|
||
### Geral | ||
|
||
- Se o branch da PR estiver "desatualizado com o branch base", não é necessário atualizá-lo continuamente: toda atualização dispara novamente os testes de CI! Geralmente, é suficiente atualizar antes de fazer o merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Se o branch da PR estiver "desatualizado com o branch base", não é necessário atualizá-lo continuamente: toda atualização dispara novamente os testes de CI! Geralmente, é suficiente atualizar antes de fazer o merge. | |
- Se a _branch_ do PR estiver desatualizada com a _branch_ base, não é necessário atualizá-lo continuamente: toda atualização dispara novamente os testes de CI! Geralmente, é suficiente atualizar apenas um pouco antes de fazer o merge. |
|
||
- Se o branch da PR estiver "desatualizado com o branch base", não é necessário atualizá-lo continuamente: toda atualização dispara novamente os testes de CI! Geralmente, é suficiente atualizar antes de fazer o merge. | ||
- PRs de pessoas que não são mantenedoras **nunca** devem atualizar submódulos git. Isso pode acontecer por engano. Informe ao autor que não há problema, será corrigido antes do merge, mas que no futuro deve usar um fork atualizado. | ||
- Se o contribuidor estiver com problemas para assinar o CLA ou usou o email errado em algum commit, solicite a correção ou o rebase da PR. No pior dos casos, feche e reabra a PR para disparar nova verificação do CLA. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Se o contribuidor estiver com problemas para assinar o CLA ou usou o email errado em algum commit, solicite a correção ou o rebase da PR. No pior dos casos, feche e reabra a PR para disparar nova verificação do CLA. | |
- Se o contribuidor estiver com problemas para assinar o CLA ou usou o email errado em algum commit, solicite a correção ou o rebase do PR. No pior dos casos, feche e reabra o PR para disparar nova verificação do CLA. |
|
||
PRs criadas por bots seguem estas práticas: | ||
|
||
- PRs que atualizam versões no registro podem ser corrigidas, aprovadas e mescladas imediatamente. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mais comum não?
- PRs que atualizam versões no registro podem ser corrigidas, aprovadas e mescladas imediatamente. | |
- PRs que atualizam versões no registro podem ser corrigidas, aprovadas e mergeados imediatamente. |
|
||
Esta página inclui diretrizes e algumas práticas comuns utilizadas por aprovadores e mantenedores. | ||
|
||
## Integração (Onboarding) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lembrar de Adicionar as âncoras
## Integração (Onboarding) | |
## Integração (Onboarding) {#onboarding} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Obrigada pela sua contribuição! Adicionei alguns comentários, antes que possa ser aprovado
|
||
Quando um colaborador assume um papel com mais responsabilidade na documentação (como aprovador ou mantenedor), ele será integrado pelos aprovadores e mantenedores existentes: | ||
|
||
- Ele será adicionado ao grupo `docs-approvers` (ou `docs-maintainers`). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Ele será adicionado ao grupo `docs-approvers` (ou `docs-maintainers`). | |
- A pessoa será adicionada ao grupo `docs-approvers` (ou `docs-maintainers`). |
Quando um colaborador assume um papel com mais responsabilidade na documentação (como aprovador ou mantenedor), ele será integrado pelos aprovadores e mantenedores existentes: | ||
|
||
- Ele será adicionado ao grupo `docs-approvers` (ou `docs-maintainers`). | ||
- Será adicionado aos canais de Slack `#otel-comms`, `#otel-maintainers` e canais internos privados. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Será adicionado aos canais de Slack `#otel-comms`, `#otel-maintainers` e canais internos privados. | |
- A pessoa será adicionada aos canais de Slack `#otel-comms`, `#otel-maintainers` e canais internos privados. |
|
||
- Ele será adicionado ao grupo `docs-approvers` (ou `docs-maintainers`). | ||
- Será adicionado aos canais de Slack `#otel-comms`, `#otel-maintainers` e canais internos privados. | ||
- Será solicitado a se inscrever nos convites de calendário para as reuniões do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Será solicitado a se inscrever nos convites de calendário para as reuniões do | |
- A pessoa será solicitada a se inscrever nos convites de calendário para as reuniões do |
- Ele será adicionado ao grupo `docs-approvers` (ou `docs-maintainers`). | ||
- Será adicionado aos canais de Slack `#otel-comms`, `#otel-maintainers` e canais internos privados. | ||
- Será solicitado a se inscrever nos convites de calendário para as reuniões do | ||
[SIG Comms](https://groups.google.com/a/opentelemetry.io/g/calendar-comms) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[SIG Comms](https://groups.google.com/a/opentelemetry.io/g/calendar-comms) | |
[SIG de Comunicações](https://groups.google.com/a/opentelemetry.io/g/calendar-comms) |
- Será adicionado aos canais de Slack `#otel-comms`, `#otel-maintainers` e canais internos privados. | ||
- Será solicitado a se inscrever nos convites de calendário para as reuniões do | ||
[SIG Comms](https://groups.google.com/a/opentelemetry.io/g/calendar-comms) | ||
e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e | |
e das reuniões de |
|
||
PRs com mudanças em traduções devem ter duas aprovações: uma de aprovador de docs e outra de aprovador de tradução. As mesmas práticas sugeridas para PRs co-gerenciadas se aplicam aqui. | ||
|
||
### Fazendo merge das PRs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
### Fazendo merge das PRs | |
### Fazendo merge das PRs {#merging-prs} |
|
||
### Fazendo merge das PRs | ||
|
||
Mantenedores podem seguir este fluxo para mesclar PRs: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mantenedores podem seguir este fluxo para mesclar PRs: | |
Mantenedores podem seguir este fluxo para dar merge nos PRs: |
|
||
Mantenedores podem seguir este fluxo para mesclar PRs: | ||
|
||
- Verifique se a PR tem todas as aprovações e todos os testes de CI passaram. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Verifique se a PR tem todas as aprovações e todos os testes de CI passaram. | |
- Verifique se o PR tem todas as aprovações e todos os testes de CI passaram. |
Mantenedores podem seguir este fluxo para mesclar PRs: | ||
|
||
- Verifique se a PR tem todas as aprovações e todos os testes de CI passaram. | ||
- Se o branch estiver desatualizado, atualize via UI do GitHub. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Se o branch estiver desatualizado, atualize via UI do GitHub. | |
- Se a branch estiver desatualizada, atualize via UI do GitHub. |
|
||
- Verifique se a PR tem todas as aprovações e todos os testes de CI passaram. | ||
- Se o branch estiver desatualizado, atualize via UI do GitHub. | ||
- Isso disparará novamente os testes de CI. Aguarde os resultados ou use o seguinte comando para rodar e mesclar em segundo plano: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Isso disparará novamente os testes de CI. Aguarde os resultados ou use o seguinte comando para rodar e mesclar em segundo plano: | |
- A atualização disparará os testes de CI para serem executados novamente. Aguarde que todos os testes passem ou use o seguinte comando para rodar os testes e dar merge em segundo plano: |
Saiba como aprovadores e mantenedores gerenciam issues e contribuições. | ||
weight: 999 | ||
cSpell:ignore: chalin Comms docsy onboarded | ||
--- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
está faltando o valor para default_lang_commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Arbythecoder aqui tem uma explicação sobre.
https://opentelemetry.io/docs/contributing/localization/#adding-default_lang_commit-to-new-pages
https://vasconcellos.dev/posts/2024-07-26-guia-contribuicao-otel-docs-pt
Se precisa de ajuda só chamar no slack.
/fix:fix:dict |
Você tem alguns problemas de formatação, roda esse comando localmente tbm: |
--- | ||
|
||
Esta página inclui diretrizes e algumas práticas comuns utilizadas por aprovadores e mantenedores. | ||
|
||
## Integração (Onboarding) | ||
## Integração (Onboarding) {#onboarding} <!-- ADDED ANCHOR HERE --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please, do not add comments explaining the changes you did. This is valid for this and for all other comments you added below
@Arbythecoder when someone makes a suggestion, you will see an option like this image you can simply click on |
@Arbythecoder you made a mistake on your last commit and added changes to the english file, instead of the portuguese one |
24c0e45
to
120b749
Compare
@Arbythecoder looks like you made some updates to the portuguese, but look here into the changes files, you still replaced the english file with the portuguese one |
Closes #6860
Description
Added Portuguese translation of the SIG Practices document as requested in localization issue #6860.
Changes
sig-practices.md