Dans l'examen du code, devrais-je demander de faire un refacteur en dehors de la portée dans une demande de traction?
J'ai étudié les meilleures pratiques pour un examen de code et je me demandais quoi faire dans le scénario suivant:
Lors d'une revue de code, je vois des améliorations potentielles, mais décide qu'ils sont en dehors de la portée de la demande de traction (PR). Devrais-je demander à la critique pour faire le refacteur dans ce même PR, ou devrais-je le différer à un futur PR puisqu'il est techniquement hors de portée?
Je pense que tous les PR devraient s'efforcer d'améliorer la qualité globale du code, car cela tend à mieux rendre l'ensemble du projet. Est-ce une mauvaise pensée? Devrais-je être plus conscient de la réduction de la portée de mes critiques de code?
Vous ne devriez pas, sauf si vous vous sentez incapable d'examiner le code tel qu'il est. Vous ne créez que des travaux inutiles. Et si vous vous sentez incapable d'examiner le code, vous devriez peut-être apprendre de nouveaux outils. Vous créez une frustration sans bonne raison et vous ne devriez pas être surpris si votre propre code aura des problèmes de passage d'un examen, en fonction des caractères impliqués.