DEV Community

Cover image for Request Changes: você usa com frequência?
Weverton Timoteo
Weverton Timoteo

Posted on

2 3

Request Changes: você usa com frequência?

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!

Image of Datadog

Create and maintain end-to-end frontend tests

Learn best practices on creating frontend tests, testing on-premise apps, integrating tests into your CI/CD pipeline, and using Datadog’s testing tunnel.

Download The Guide

Top comments (0)

Billboard image

The Next Generation Developer Platform

Coherence is the first Platform-as-a-Service you can control. Unlike "black-box" platforms that are opinionated about the infra you can deploy, Coherence is powered by CNC, the open-source IaC framework, which offers limitless customization.

Learn more