Hé! Je m'appelle Sergey Zagursky, je travaille chez Joom dans l'équipe infrastructure. Dans ma pratique en tant que réviseur de code, je rencontre régulièrement le fait que l'auteur ne comprend pas que le réviseur n'est pas une boîte noire magique dans laquelle vous pouvez apporter des modifications et obtenir des commentaires à leur sujet. Le critique, comme l'auteur, en tant qu'être humain, a un certain nombre de faiblesses. Et l'auteur devrait (si, bien sûr, il est intéressé par un examen de la qualité), aider le critique autant que possible.
Je veux vous dire comment l'auteur du code peut simplifier le travail du réviseur et ainsi augmenter à la fois la qualité de la révision et la productivité du réviseur. Cet article peut très bien être utilisé dans la documentation interne de votre entreprise comme un guide pour préparer les modifications à réviser. Il a, en fait, été compilé à partir d'un tel guide.
Pourquoi faisons-nous une revue de code
. , .
. , . , . . . , « », . , .
: . — . , , . . , . , , .
, . , . , , write only . .
, . - ( ). , .
— .
, , , . , . . , , « ». , , . . , . - .
, . . , . , . , . .
- , , . , , , , . , , , , , .
, -, . , , , .
. , , -, , . . .
, , , , . Pull Request .
, , , . , , .
, Pull Request, , . 500 . — 1000 . , 1000 , Pull Request’.
, , , . , , . 1000+ , , , . . , . , Pull Request’, « ».
, sed’. . . Pull Request’, , .
. , : , , , . . — . , , .
- , , . . , « » - . . , , , , .
, , . , , . , .
, . ? ! , . , . . . . - , , TODO-, , . , . .
, 10% , , . , , . , 20%, 50% , .
.
Pull Request’. , . , . «OK» «», . , , . , . — . - , , . , , , . , , . , , , .
Pull Request, , . . rebase .
? , !