Ao revisar Pull Requests podemos bloquear o merge de acontecer, ao utilizar a opção Request Changes
(solicitar mudanças). Mas é uma boa prática utilizá-lo?
A boa prática de uso do Request Changes
é quando você encontra alguma modificação que, por alguma razão, não foi encontrada na suite de testes porém se aquele PR for mergeado, irá quebrar a aplicação no ambiente de produção.
Cuidado com o poder
Request Changes também é uma forma de poder, pense que ao utilizá-la você está bloqueando o Pull Request da(s) pessoa(s) que trabalharam nele.
Antes de chegar nesse tipo de prática eu recomendo refletir se as pessoas autoras envolvidas costumam atender aos comentários realizados em PRs de suas autorias.
Caso atender aos comentários seja uma prática do time, não necessariamente resolver (respondendo de alguma maneira), talvez apenas fazer um review sem conceder a aprovação seja suficiente. ✅
É tudo sobre comunicação
Caso você não tenha outra maneira de se comunicar com seu time e precise recorrer ao Request Changes
com frequência, você deve trabalhar em melhorar a comunicação.
💬 Ter uma boa comunicação é essencial para a colaboração acontecer de maneira efetiva!
Top comments (0)