Fluxo de trabalho colaborativo Git

🔄 O que é um Pull Request (ou Merge Request no GitLab)?

Um Pull Request (PR) é quando você diz:

"Oi, eu fiz uma alteração na branch minha-feature, gostaria que essa mudança fosse revisada e mesclada (mergeada) com a branch principal, geralmente a main ou develop."

Ele serve para:

• Revisar o código antes do merge
• Discutir mudanças com a equipe
• Garantir que os testes passem (CI/CD)
• Manter o histórico limpo e rastreável

📋 Fluxo típico de trabalho com tickets + PR:

1. Pegue um ticket (ou issue)

• Um ticket descreve um problema ou funcionalidade.
• Ex: "Criar tela de login", "Corrigir bug na busca de produtos", etc.

Geralmente ele tem:

• Descrição do que precisa ser feito
• Critérios de aceitação
• Prioridade

2. Crie uma branch para esse ticket

Usa um nome descritivo e/ou com ID do ticket:

git checkout -b feat/login-form

Convenções comuns:
feat/, fix/, chore/, hotfix/, etc.

3. Faça as alterações e comite com mensagens claras

git add .
git commit -m "feat: adiciona formulário de login"

🧠 Dica: siga boas práticas de commit. Ex: fix: corrige bug no cálculo de descontos refactor: melhora performance da consulta

4. Suba a branch pro GitLab

git push -u origin feat/login-form

5. Crie um Merge Request (Pull Request) no GitLab

• Título descritivo: feat: formulário de login
• Descreva o que foi feito
• Relacione com o ticket (ex: Closes #123)
• Adicione reviewers (colegas que vão revisar)
• Escolha a branch base (geralmente main ou develop)

🧠 O que ficar atento ao fazer PR:

✅ Antes de abrir o PR:

• O código está testado?
• Seguiu a convenção da equipe (nomes, commits, estilo)?
• Rodou os testes e passou?
• Fez rebase com main ou develop pra evitar conflitos?

  git fetch origin
  git rebase origin/main
  

✅ Durante o PR:

• Descreva claramente o que o código faz
• Se possível, adicione screenshots ou vídeos (UI)
• Marque os responsáveis pela revisão
• Responda aos comentários com respeito e clareza

✅ Depois do PR aprovado:

• O revisor pode aprovar e fazer o merge
• Ou você pode fazer o merge, dependendo da política da equipe

💡 Extras

💬 Comunicar com clareza:

• Explique por que a mudança foi feita (não só o que mudou)
• Evite mensagens genéricas como "arrumei bug" — detalhe o problema e a solução

🔍 Sempre revise seu próprio código:

• Evite código morto, console.log, prints de debug
• Verifique nomes de variáveis e legibilidade