Fornecer uma revisão de pedido de puxar¶
Todos são bem-vindos para rever qualquer contribuição para o projeto BeeWare. Há algumas considerações importantes que devem ser tidas em conta antes de começar.
PENSAR antes de rever¶
Antes de se envolver numa avaliação, PENSE. Como revisores, devemos considerar se a resposta que estamos prestes a enviar é:
- Verdadeira. Esforce-se sempre para fornecer sugestões e informações precisas.
- Útil. Estamos a fornecer orientação sobre como melhorar a submissão; essa orientação deve identificar claramente a origem de um problema ou de um caso de uso não considerado e, idealmente, fornecer um caminho a seguir para resolver ou satisfazer a preocupação.
- Inspiradora. Cabe a nós inspirar o autor a querer trabalhar com as nossas alterações solicitadas.
- Necessária. A expectativa é que o autor leia tudo o que postamos; devemos respeitar seu tempo e esforço postando só quando necessário.
- Gentil. Há várias maneiras de apresentar o mesmo retorno de informação; precisamos garantir que estamos a escolher ser gentis, solidários e construtivos com as nossas palavras.
É perfeitamente possível PENSAR e, ao mesmo tempo, fazer uma revisão eficaz. Os conceitos discutidos acima não impedem que aponte quaisquer problemas que encontrar num PR. Os colaboradores não terão a oportunidade de melhorar a sua contribuição se não tiverem conhecimento das áreas que precisam ser melhoradas. O importante é estar ciente de como está a apresentar esse retorno de informação. Tente despersonalizar a sua avaliação. Em vez de dizer: "Você cometeu um erro", pode dizer: "Este código pode ser melhorado". Analise o código, não o autor.
É importante lembrar-se de fornecer retorno de informação positivo, além de identificar as áreas que precisam ser melhoradas. Se, por exemplo, as alterações forem especialmente úteis, fizerem algo particularmente inteligente ou for apresentado a uma API que não conhecia, informe o autor! Nunca subestime o efeito de apontar algo que alguém fez corretamente ou bem, no meio de uma situação em que tudo o mais que apontou são problemas que precisam ser resolvidos.
Sugestões de revisão do GitHub¶
A interface de revisão do GitHub tem um mecanismo para sugestões de alteração, no qual pode fornecer a alteração exata que está a sugerir como uma substituição do conteúdo existente. Lembre-se de que, até que sejam aceites e confirmadas, essas alterações sugeridas não passarão pelas verificações de pré-compromisso e de análise de código. Portanto, esta funcionalidade deve ser usada para alterações menores, pois quanto maior for a alteração sugerida, maior será a probabilidade dela introduzir problemas.