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

Default template pull requests in the pipeline #867

Merged
merged 5 commits into from
Oct 22, 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
64 changes: 64 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,64 @@

# Template Pull Requests - Pipeline

## 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**:
- [ ] **[Feature]**: Para novas funcionalidades.
- [ ] **[Data]**: Para subida de novos dados em produção.
- [ ] **[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.
- [ ] **[Deactivate]**: Para desativar o schedule da Pipeline
---
- 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 na pipeline, scripts ou modelo de dados utilizado.

- **Principais alterações na pipeline/scripts:** <!-- Cite as principais mudanças na pipeline/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 localmente
- [ ] Testado na Cloud

**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
tricktx marked this conversation as resolved.
Show resolved Hide resolved


## 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