Este documento fornece instruções sobre como analisar uma mudança. Não se esqueça dos critérios abaixo para que uma mudança seja válida:
- Essa mudança se encaixa no propósito declarado do projeto?
- Essa mudança é válida dentro da arquitetura atual do projeto?
- Essa mudança introduz falhas de design que vão causar problemas no futuro?
- Essa mudança segue as práticas recomendadas estabelecidas para o projeto?
- Essa mudança é uma boa maneira de executar a função descrita?
- Essa mudança introduz algum risco de segurança ou instabilidade, ou corrompe algum app atual?
Para mais informações sobre como analisar uma mudança, acesse a seção Como analisar uma mudança (em inglês) da documentação do Gerrit.
Ao ser designado como avaliador de uma mudança, você vai receber um email com um link para ela. Siga estas etapas para analisar a mudança:
Clique no link do e-mail para a mudança. A tela de Informações da mudança do Gerrit será aberta. Essa tela fornece uma descrição, identifica o colaborador da mudança e todos os avaliadores, e fornece uma lista dos arquivos associados a ela.
Leia a descrição e entenda qual é o objetivo da mudança.
Clique em um arquivo para analisar as mudanças no arquivo. O arquivo abre em uma tela de comparação de dois painéis com o arquivo atual à esquerda e o arquivo com as mudanças propostas à direita. Para analisar o arquivo:
Use o mouse para destacar (clique e arraste) a parte do código para a qual você tem um comentário ou dúvida. A mensagem "Pressione c para comentar" aparecerá.
Pressione c. Uma caixa de comentários aparecerá.
Digite um comentário ou dúvida e clique em SAVE.
Clique no número exclusivo da mudança (canto superior esquerdo) para voltar à tela de informações da mudança.
Continue analisando cada arquivo na mudança.
Quando terminar de analisar todos os arquivos, volte para a tela de informações da mudança e clique em REPLY. A tela "Submit requirement votes" aparecerá.
Escolha uma destas pontuações para a mudança:
- +1 Parece bom para mim, mas outra pessoa também precisa aprovar.
0 Sem pontuação
-1 Prefiro que não seja enviada no estado em que se encontra. Há correções a serem feitas.
-2 Essa mudança não deve ser enviada.
Clique em REPLY para enviar seus comentários e pontuação para o desenvolvedor da mudança.
Se há problemas a serem corrigidos, o desenvolvedor fará isso e pedirá que você analise as correções. Caso contrário, o processo de análise da mudança será concluído.