... | ... | @@ -41,6 +41,25 @@ Dessa forma, o fluxo usual de trabalho é o seguinte: |
|
|
7. mover o card da task no Trello para a lista _Code Review_ e avisar no Slack (canal #code-review) que a task está pronta para review;
|
|
|
8. fim! 😃
|
|
|
|
|
|
### Padrões de commit
|
|
|
|
|
|
Quando realizar um commit, tente manter o seguinte padrão nas mensagens:
|
|
|
|
|
|
- título: número da task e breve descrição da mudança;
|
|
|
- se preciso, adicione um descrição mais longa da mudança, do porquê ela se faz necessária ou do porquê foi feito dessa forma; essa descrição longa deve estar separada do título por uma linha em branco;
|
|
|
- se mais de uma pessoa participou das mudanças do commit: indique a co-autoria conforme exemplo abaixo, também separada por uma linha em branco da descrição longa (o Git precisa de um formato bem específico nessa parte, com uma pessoa por linha; não é preciso adicionar uma linha para a pessoa que está fazendo o commit, porque o Git já pega isso automaticamente).
|
|
|
|
|
|
Exemplo:
|
|
|
|
|
|
```
|
|
|
#48: Fix patient name field placeholder
|
|
|
|
|
|
The placeholder was clipped smartphones with small screens; reduced placeholder
|
|
|
length for better UX on those devices.
|
|
|
|
|
|
Co-authored-by: Marcos Silva <[email protected]>
|
|
|
```
|
|
|
|
|
|
## Code Review
|
|
|
|
|
|
Após o desenvolvimento de uma task, um _pull/merge request_ (PR) deve ser aberto com destino à branch _develop_ do repositório relativo à task: _app_ ou _api_. Todos os PRs são revisados por pelo menos dois AGES III, que se responsabilizam por garantir a qualidade do que foi desenvolvido e que os artefatos e estruturas se adequem aos padrões definidos neste documento.
|
... | ... | |