Se você acha que o código deve ser corrigido antes de mesclar, faça comentários. De preferência com "por que" para que o desenvolvedor possa aprender.
Lembre-se de que o código é lido com muito mais frequência do que escrito. Então, coisas que parecem "menores" podem realmente ser realmente importantes (nomes de variáveis, por exemplo).
No entanto, se você estiver fazendo comentários que parecem tediosos, talvez considere:
- O seu processo de CI deve detectar isso?
- Você tem um "guia do desenvolvedor" claro para fazer referência (ou está tudo documentado na sua cabeça)?
- Esses comentários realmente contribuem para a qualidade do código?
Muitas pessoas sacrificam a produtividade no altar do processo ou da perfeição. Tenha cuidado para não fazer isso.
Tente visitar seu colega pessoalmente, se possível. Ou use videochamadas. Construir um relacionamento torna as críticas (mesmo as revisões de código) mais fáceis de gerenciar.
Se você achar que um trecho de código tem muitos problemas, solicite a revisão em pequenos pedaços de código. Mudanças incrementais têm maior probabilidade de evitar alguns dos problemas de design mais significativos, porque são, por definição, menores.
Mas absolutamente não mescle as coisas e depois volte e corrija. Isso é passivo agressivo e se o desenvolvedor achar que você está fazendo isso, você vai matar o moral deles.