-
Notifications
You must be signed in to change notification settings - Fork 4
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
create template pr in queries-basedosdados
- Loading branch information
Showing
1 changed file
with
63 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,63 @@ | ||
|
||
# Template Pull Requests - Queries-basedosdados | ||
|
||
## Nomeação do Pull Request | ||
|
||
A nomeação de cada Pull Request (PR) deve seguir o seguinte padrão: | ||
|
||
- O título de cada Pull Request (PR) deve começar com uma das seguintes palavras-chave, entre colchetes. Além disso, **marque a palavra-chave que melhor descreve o seu PR atual**: | ||
- [ ] **[Dbt]**: Para subida de novos dados em produção. | ||
- [ ] **[Feature]**: Para novas funcionalidades. | ||
- [ ] **[Table-Approve]**: Para rodar apenas o Table-Approve. | ||
- [ ] **[Bugfix]**: Para correções de bugs. | ||
- [ ] **[Refactor]**: Para mudanças no código que não alteram a funcionalidade. | ||
- [ ] **[Docs]**: Para atualizações na documentação. | ||
- [ ] **[Test]**: Para mudanças relacionadas a testes. | ||
- [ ] **[Chore]**: Para tarefas menores e de manutenção. | ||
|
||
--- | ||
- Exemplos de título: | ||
- **[docs] br_me_caged** | ||
- **[Feature] br_cgu_servidores_publicos** | ||
|
||
## Draft: | ||
- Ao abrir o PR, deverá coloca-lo como draft | ||
|
||
|
||
## Descrição do PR: | ||
- Explique de maneira clara e concisa o objetivo deste PR. O que foi alterado? Qual o problema que ele resolve? | ||
- **Motivação/Contexto:** <!-- Qual a necessidade dessa mudança? --> | ||
|
||
|
||
## Detalhes Técnicos: | ||
- Detalhe as mudanças mais técnicas, como ajustes no dbt, scripts ou modelo de dados utilizado. | ||
- **Principais alterações no dbt/scripts:** <!-- Cite as principais mudanças no dbt/script --> | ||
- **Mudanças nos dados e no schema:** <!-- Cite se há mudanças nos dados e no schema --> | ||
- **Impacto no desempenho:** <!-- Cite os impactos de desempenho dessas mudanças --> | ||
|
||
- Se alguma parte do código precisar de alguma atenção a mais, comente na linha sinalizando para os revisores. | ||
|
||
## Teste e Validações: | ||
|
||
- Relate os testes e validações relacionado aos dados/script: | ||
- [ ] Testado em `queries-basedosdados-dev` | ||
- [ ] **Decisões relacionadas aos testes:** <!-- Cite as decisões relacionadas ao teste --> | ||
|
||
- **Caso haja algo relacionado aos testes que vale a pena informar:** <!-- Cite informações sobre os testes? --> | ||
|
||
## Riscos e Mitigações: | ||
- Identifique os riscos potenciais desta mudança e como mitigar esses Riscos | ||
|
||
- Riscos conhecidos: <!-- Cite problemas que podem surgir --> | ||
- Planos de rollback: <!-- Explique como reverter as mudanças --> | ||
|
||
## Dependencias: | ||
- Liste quaisquer dependências externas, como bibliotecas, outros PRs ou mudanças que precisam ser feitas antes deste merge. | ||
- [ ] Dependências: <!-- Cite dependencias, bibliotecas e outros PRs que são relacionados a esse Pull Requests antes de mergear --> | ||
- [ ] Nenhuma dependencias adicional | ||
|
||
|
||
## Revisadores: | ||
- Quando o PR estiver pronto para ser revisado, retire o **Draft** através do **Ready for reviews**, marque os revisadores de repositório, envie o PR no nosso [discord](https://discord.gg/V3yTWRYWZZ) na aba **Correções de PRs, arquiteturas e afins** e marque a **@equipe_dados**: | ||
- Revisadores recomendados no github: | ||
- basedosdados/dados |