Je voulais savoir si ma façon de traiter les fichiers source à supprimer du contrôle de version pouvait être considérée comme une mauvaise pratique.
Je veux vous expliquer en vous basant sur cet exemple:
Je me suis récemment mis très en colère parce que je devais trier minutieusement les classes Java dans un programme qui était essentiellement du code mort, mais il n'était nulle part documenté et non commenté dans ces classes Java. Bien sûr, ils devaient être supprimés, mais avant que je supprime ce genre de choses redondantes, j'ai une habitude - certains diront peut-être étrange -:
Je ne supprime pas immédiatement ces fichiers redondants via SVN-> Supprimer (commande de choix du système de contrôle de version de votre système de contrôle de version), mais placez plutôt des commentaires dans ces fichiers (je renvoie à la fois au début et à la fin) auxquels ils vont accéder. être supprimé + mon nom + la date et aussi - plus important encore - pourquoi ils sont supprimés (dans mon cas, car ils étaient morts, code source de confusion). Ensuite, je les enregistre et les engage dans le contrôle de version. La prochaine fois que je devrai valider / archiver quelque chose dans le projet pour le contrôle de version, j'appuierai sur SVN-> Supprimer, puis ils seront finalement supprimés dans Version Control - toujours bien sûr restaurable grâce à des révisions et c'est pourquoi j'ai adopté cette habitude.
Pourquoi faire cela au lieu de les supprimer tout de suite?
Ma raison est, je veux avoir des marqueurs explicites au moins dans la dernière révision dans laquelle ces fichiers redondants existaient, pourquoi ils méritaient d'être supprimés. Si je les supprime tout de suite, ils sont supprimés, mais rien n’indique pourquoi ils ont été supprimés. Je veux éviter un scénario typique comme celui-ci:
"Hmm ... pourquoi ces fichiers ont-ils été supprimés? Je fonctionnais bien avant." (Presses 'revert' -> Le type qui a annulé sa visite est parti pour toujours ou n'est plus disponible dans les prochaines semaines et le prochain cessionnaire doit découvrir fastidieusement ce que sont ces fichiers.)
Mais ne notez-vous pas pourquoi ces fichiers ont été supprimés dans les messages de validation?
Bien sûr, mais un message de validation n’est parfois pas lu par des collègues. Lorsque vous essayez de comprendre le code (dans mon cas, mort), vous ne vérifiez pas d'abord le journal de contrôle de version avec tous les messages de validation associés. Au lieu de parcourir le journal, un collègue peut voir tout de suite que ce fichier est inutile. Cela lui fait gagner du temps et il / elle sait que ce fichier a probablement été restauré pour de mauvais (ou au moins cela soulève une question.