Les réviseurs doivent être objectifs.
Il est clair que vous avez formé une opinion sur le code en question avant même de l'avoir revu, et il semble que le fixateur et vous avez pris position. Si tel est le cas, alors vous aurez du mal à paraître objectif, et encore plus difficile à être objectif. Rien de tout cela n’aide le processus, et il se peut que la meilleure chose, et la plus objective que vous puissiez faire, c’est de vous retirer sous prétexte que vous êtes trop près du problème.
Considérons une approche d'équipe.
S'il n'est pas possible de supprimer vous-même, vous pouvez peut-être demander à plusieurs autres ingénieurs de réviser le code en même temps. Soit ils conviendront avec vous que le code devrait être rejeté, soit ils ne le seront pas. S'ils sont d'accord avec vous, alors ce ne sera plus juste vous contre le fixateur, et vous serez en mesure de prouver que l'équipe a examiné le correctif de manière objective et a décidé de ne pas l'accepter. Par contre, s’ils décident d’accepter le correctif, ce sera également une décision de l’équipe. Il va sans dire que vous devez participer avec la plus grande ouverture d'esprit possible et que vous ne devez pas essayer d'influencer les opinions des autres membres de l'équipe autrement que par une discussion rationnelle. Important: s'il y a un mauvais résultat plus tard, ne jette pas l'équipe sous le bus en disant : « Eh bien , je a toujours dit que c'était un mauvais code, mais les autres membres de l'équipe m'ont dépassé en nombre. "
Les rejets font naturellement partie du processus de révision du code.
Le processus de révision du code n'est pas là pour endosser les correctifs de correctifs de la part de personnes plus expérimentées; il est là pour protéger et améliorer la qualité du code. Il n'y a rien de mal à rejeter un correctif à condition que vous le fassiez pour la bonne raison, c'est-à-dire que le correctif n'améliore pas le code. Si, après un examen ouvert du code, vous estimez toujours que le correctif ne réduit pas le risque et / ou l'ampleur d'un problème pouvant être démontré, vous devez le rejeter. Ce n'est pas personnel, juste votre opinion honnête. Si le fixateur n’est pas d’accord, c’est bien aussi, et à ce moment-là, la direction a du mal à résoudre le problème. Assurez-vous simplement de rester honnête, ouvert et professionnel.
La responsabilité va dans les deux sens.
Vous avez dit que vous ne voulez pas être responsable de ce changement, apparemment parce que vous ne croyez pas qu'il y a un problème. Cependant, vous devez comprendre que si vous vous trompez et qu'il y a un problème, vous pouvez être responsable du rejet du code, ce qui l'aurait évité.
Prendre des notes.
Tenir un journal écrit du processus d’examen vous aidera à garder les faits exacts. Ecrivez vos pensées et vos préoccupations lors de l'examen, la description et les résultats de tous les tests que vous pourriez exécuter pour mesurer le problème allégué, le correctif, etc. Si le problème s'aggrave, vous aurez une trace de ce que vous avez fait pour soutenir votre position. Si le problème se reproduit à l'avenir (cela le sera probablement si le fixateur est attaché à sa propre vue), vous aurez quelque chose à rafraîchir pour votre mémoire.