Mea Maxima Culpa pour la façon dont j'ai abordé le sujet qui était très brute et agressive. Je tiens à m'excuser Bertrand si mon post a été perçu comme une attaque, ce qui n'est absolument pas mon intention.
J'ai un background différent avec mon cursus, et j'ai une approche pour un projet déjà existant qui correspond à ce background et à ce que je fais.
Cela n'excuse pas de mon message, mais cela permet de comprendre pourquoi j'ai voulu orienter la conversation sur ce sujet.
Je considère qu'un projet Open Source (OS) mérite d'être maintenu, et avec une certaine clarté / lisibilité / code quality. Ca passe par l'attribution d'un organisme d'automatisation de test, d'une Intégration et d'un Développement Continu (CI/CD),
et du nettoyage si nécéssaire du code (mort, non clair, dupliqué, etc.). C'est en plus en rapport avec le sujet de mon projet de fin d'études (artofcode.eu), qui est un analyseur statique de code, donc c'est un sujet que je porte d'autant plus dans mon coeur.
Ce que je relève donc, c'est que SonarQube soulève 3000+ "code smells", du code qui sera donc difficile à maintenir, pas clair, rébarbatif ou mort.
Bien que cela n'impacte pas le fonctionnement de Medshake, c'est en arrière-plan que cela devient compliqué: un nouveau-venu qui prend le temps d'assimiler le code métier d'un projet OS perd du temps et aura des difficultés si une classe contient une vaste quantité de méthodes et de propriétés, qu'il y a des comportements qui se répètent, ou qu'il y a du code qui, finalement, ne sera jamais exploité. Et c'est même tarif pour une personne qui participe à maintenir ce même code métier: c'est arrivé à tout le monde de revenir sur du code métier vieux de 3 ans, et de plus comprendre pourquoi une méthode ou fonction a tel comportement.
Ca me gène, mais pas dans le sens "je suis choqué", plus dans le sens ou si nous, en tant que communauté, nous nous y attaquons pas maintenant, et bien la dette technique n'en sera que plus grande. J'ai beau être un stagiaire et avoir ma todolist, j'ai aussi un intérêt personnel derrière: j'ai envie de contribuer, j'ai un grand intérêt pour l'écosystème Open Source, et j'ai déjà pu contribuer par le passé (exercism par exemple: https://exercism.io/team/mentors?page=7).
Donc bien que je sois rattaché à une entreprise, cela ne m'empêche pas de prendre à coeur ce que je souhaite faire pour le bien d'un projet et d'une communauté, même si cela est temporaire !
J'admet ne pas être encore rentré dans le détail de toute l'analyse, il y a forcément des fausses alertes (il y en a toujours), je ne sais pas exactement quel est le ratio, mais cela nous donne des pistes pour optimiser et améliorer le code métier existant, et je suis prêt à m'investir et à y passer du temps dessus pour différencier le vrai du faux !