Nouvelle publication demandée à StackOverflow:
Je travaille dans un petit temps de développement avec un temps de développement très limité. Nous développons un outil important pour le résultat de notre travail, mais non utilisé quotidiennement. Je suis la seule personne de l'équipe à avoir une formation de programmeur.
Mon problème est que j'ai poussé pour les revues de code avant de fusionner de nouveau dans le tronc pendant plus d'un an. Tout le monde était d'accord, mais ce n'est que mon code qui a été révisé. De retour de longues vacances, je reviens à un coffre avec des commentaires de code car "c'est une solution laide - à retirer le plus tôt possible" et "solution rapide". Ce qui est également nouveau, c'est qu'un gars a été nommé responsable de l'outil. (Un rôle qui m'a été proposé en premier mais que j'ai refusé pour une raison non liée au travail.) Et il pense que c'est une bonne façon de travailler: comme nous avons un temps si limité à développer, nous devrions couper les coins comme ça.
Ma préoccupation est que les autres développeurs écrivent du code laid: cassant souvent l'encapsulation, écrivant d'énormes classes, ajoutant des classes internes à des endroits étranges, ayant peu ou pas de tests unitaires, etc. Il sera finalement impossible de développer davantage l'outil.
Dois-je insister pour que nous effectuions des révisions de code avant de fusionner de nouveau dans le tronc ou suis-je juste une chienne de qualité de code?