Skip to content

Commit ae4104b

Browse files
(#474) - Adiciona melhores práticas de título e descrição de RP aos arquivos CONTRIBUTING (#530)
* Atualizar a seção 13 - Crie um PR em Ingles * Atualizar a seção 13 - Crie um PR em Portugues * Update CONTRIBUTING.md Co-authored-by: Camila Maia <cmaiacd@gmail.com> * Update CONTRIBUTING.md Co-authored-by: Camila Maia <cmaiacd@gmail.com> * Adicionando melhores exemplos checklist * Update CONTRIBUTING_EN.md * Pequenos detalhes no CONTRIBUTING.md * Pequenos detalhes CONTRIBUTING_EN.md --------- Co-authored-by: Camila Maia <cmaiacd@gmail.com>
1 parent c6cb92a commit ae4104b

2 files changed

Lines changed: 82 additions & 6 deletions

File tree

CONTRIBUTING.md

Lines changed: 39 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -580,7 +580,45 @@ _Este conteúdo é baseado no [site do keep a changelog][keep-a-changelog], já
580580

581581
### 13. Crie um PR no GitHub
582582

583-
[Crie um PR no GitHub][github-creating-a-pr].
583+
[Crie um PR no GitHub][github-creating-a-pr] para enviar suas alterações para revisão. Para garantir que seu Pull Request (PR) seja claro, eficaz e revisado rapidamente, siga estas boas práticas:
584+
585+
#### Escreva um Título Descritivo para o PR
586+
- Use títulos claros e específicos para descrever o propósito das suas alterações. Um bom título ajuda às pessoas mantenedoras a entender a intenção do PR rapidamente e melhora a rastreabilidade do projeto.
587+
- **Exemplo**: Em vez de “Corrigir problema”, use “Adiciona utilitário `convert_uf_to_text` para lidar com códigos de estados brasileiros.”
588+
- **Benefícios**:
589+
- Títulos claros facilitam a priorização e o entendimento pelos revisores.
590+
- Melhoram a organização e a busca no projeto.
591+
592+
#### Forneça uma Descrição Detalhada do PR
593+
- Inclua uma descrição completa no seu PR para explicar:
594+
- **O que** foi feito (ex.: adicionou uma nova função, corrigiu um bug).
595+
- **Por que** foi feito (ex.: para resolver uma issue específica ou melhorar o desempenho).
596+
- **Quais problemas** foram resolvidos ou melhorias aplicadas (ex.: referencie a issue ou descreva a melhoria).
597+
- **Exemplo**:
598+
Este PR adiciona o utilitário convert_uf_to_text para converter códigos de estados brasileiros (ex.: “SP”) em nomes completos (ex.: “São Paulo”). Resolve a issue #474, melhorando a reutilização de código para formatação de endereços. A função inclui validação de entrada e testes atualizados.
599+
- **Benefícios**:
600+
- Descrições detalhadas agilizam o processo de revisão ao fornecer contexto.
601+
- Ajudam futuros mantenedores a entender o propósito e o histórico do código.
602+
603+
#### Vincule o PR à Issue Relacionada
604+
- Referencie a issue que seu PR resolve usando palavras-chave como `Closes #474` ou `Fixes #474` na descrição do PR. Isso fecha a issue automaticamente quando o PR for mesclado.
605+
- **Exemplo**: `Closes #474`
606+
- **Benefícios**:
607+
- Vincular issues mantém o repositório organizado e garante o rastreamento de tarefas.
608+
- Automatiza o fechamento de issues, reduzindo trabalho manual para mantenedores.
609+
- Para mais detalhes, consulte a [documentação do GitHub sobre fechamento automático de issues](https://docs.github.com/pt/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue).
610+
611+
#### Verifique o Template de Descrição do PR
612+
- Certifique-se de que seu PR segue o template de descrição do repositório. Verifique todos os itens obrigatórios, como cobertura de testes, atualizações de documentação ou entradas no changelog.
613+
- **Exemplo de Checklist**: (mostrando como fica quando preenchido):
614+
- [x] Alterações no código foram testadas.
615+
- [x] Documentação (READMEs) foi atualizada.
616+
- [ ] Entrada no changelog foi adicionada (marque apenas se aplicável).
617+
- **Nota sobre a Sintaxe**:
618+
- Use [x] para marcar itens concluídos e [ ] para itens não concluídos, sem espaços dentro dos colchetes (ex.: [ x ] ou [x ] não será renderizado corretamente no GitHub).
619+
- **Benefícios**:
620+
- Seguir o template garante que o PR esteja completo e pronto para revisão.
621+
- Reduz a necessidade de idas e vindas com revisores, acelerando o processo de mesclagem.
584622

585623
### 14. Atualizar a Sua Branch se Necessário
586624

CONTRIBUTING_EN.md

Lines changed: 43 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ How to make your first contribution:
1919
- [8. Make your Changes](#8-make-your-changes)
2020
- [9. Test your Changes](#9-test-your-changes)
2121
- [10. Update READMEs](#10-update-readmes)
22-
- [11. Commit and push your Changes](#10-commit-and-push-your-changes)
23-
- [12. Add changelog Entries](#11-add-changelog-entries)
24-
- [13. Create a GitHub PR](#12-create-a-github-pr)
25-
- [14. Update your branch if Needed.](#13-update-your-branch-if-needed)
22+
- [11. Commit and push your Changes](#11-commit-and-push-your-changes)
23+
- [12. Add changelog Entries](#12-add-changelog-entries)
24+
- [13. Create a GitHub PR](#13-create-a-github-pr)
25+
- [14. Update your branch if Needed.](#14-update-your-branch-if-needed)
2626

2727
### 1. Create a GitHub Account
2828

@@ -540,7 +540,45 @@ This content is based on the [Keep a Changelog][keep-a-changelog] site, as we fo
540540
541541
### 13. Create a GitHub PR
542542
543-
[Create a GitHub PR][github-creating-a-pr].
543+
[Create a GitHub PR][github-creating-a-pr] to submit your changes for review. To ensure your Pull Request (PR) is clear, effective, and reviewed quickly, follow these best practices:
544+
545+
#### Write a Descriptive PR Title
546+
- Use clear and specific titles to describe the purpose of your changes. A good title helps maintainers understand the PR’s intent at a glance and improves project traceability.
547+
- **Example**: Instead of “Fix issue,” use “Add utility `convert_uf_to_text` to handle Brazilian state codes.”
548+
- **Benefits**:
549+
- Clear titles make it easier for reviewers to prioritize and understand the PR.
550+
- They improve the project’s organization and searchability.
551+
552+
#### Provide a Detailed PR Description
553+
- Include a comprehensive description in your PR to explain:
554+
- **What** was done (e.g., added a new function, fixed a bug).
555+
- **Why** it was done (e.g., to address a specific issue or improve performance).
556+
- **What issues** were resolved or improvements applied (e.g., link to the issue or describe the enhancement).
557+
- **Example**:
558+
This PR adds the convert_uf_to_text utility to convert Brazilian state codes (e.g., "SP") to full state names (e.g., "São Paulo"). It addresses issue #474 by improving code reusability for address formatting. The function includes input validation and updated tests.
559+
- **Benefits**:
560+
- Detailed descriptions speed up the review process by providing context.
561+
- They help future maintainers understand the code’s purpose and history.
562+
563+
#### Link the PR to the Related Issue
564+
- Reference the issue your PR addresses using keywords like `Closes #474` or `Fixes #474` in the PR description. This automatically closes the issue when the PR is merged.
565+
- **Example**: `Closes #474`
566+
- **Benefits**:
567+
- Linking issues keeps the repository organized and ensures tasks are tracked.
568+
- It automates issue closure, reducing manual work for maintainers.
569+
- For more details, see the [GitHub documentation on closing issues automatically](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue).
570+
571+
#### Verify the PR Description Template
572+
- Ensure your PR follows the repository’s PR description template (if provided). Check all required items, such as test coverage, documentation updates, or changelog entries.
573+
- **Example Checklist**: (showing how it looks when completed):
574+
- [x] Code changes are tested.
575+
- [x] Documentation (READMEs) is updated.
576+
- [ ] Changelog entry is added.
577+
- **Syntax Note**:
578+
- Use [x] to mark completed items and [ ] for incomplete ones, with no spaces inside the brackets (e.g., [ x ] or [x ] will not render correctly on GitHub).
579+
- **Benefits**:
580+
- Adhering to the template ensures your PR is complete and ready for review.
581+
- It reduces back-and-forth with reviewers, speeding up the merge process.
544582
545583
### 14. Update your branch if needed.
546584

0 commit comments

Comments
 (0)