Cette liste de remarques est plutôt un historique de mon investigation, certaines choses ne sont pas pertinentes (par exemple le slash ne posait pas problème). L’important est la MR et ma remarque plus bas sur le processus de relecture.
Pour clarifier, puisque mon ton est peut-être ambigu :
- je ne veux pas critiquer les erreurs dans le code, j’en ai faites aussi, et la multiplicité et la complexité des outils n’aide pas. Seulement que je pense que sur Duniter particulièrement la relecture devrait être plus lente et rigoureuse.
- Peut-être que si le relecteur a contribué dans la durée à la MR, alors une tierce personne (même non-experte) devrait relire aussi. Ainsi les bugs comme un renommage non-exhaustif seraient détectés. (comme quand on rédige un texte et qu’on ne voit plus les fautes à force de le relire en boucle)
- En consolidant la relecture, j’espère éviter qu’on puisse se prendre des remarques qui auraient dû être faites pendant la relecture 6 mois plus tôt. (autant les miennes que les tiennes)