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

[Feature] Create Template PR Default #805

Merged
merged 2 commits into from
Oct 31, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
63 changes: 63 additions & 0 deletions .github/pull_request_template.md
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
Loading