22 votes

Extrait de changements pertinents pour l'examen du code

Je suis en train de travailler sur un projet avec une équipe où nous arriver tôt et souvent. Personne archivages sont terminées à la satisfaction de le développeur de faire le changement (y compris les tests si possible), mais parfois le sens du travail change légèrement précédente et s'engage besoin d'être refaites et faire d'une autre manière. Ou, stub code est rempli au plus tard, s'engage.

Quand vient le temps pour l'examen du code, il existe une séquence de commits étiquetés avec le même bug id de suivi nombre. Il est facile d'obtenir une liste de ces changements. Lorsqu'un examinateur regarde à travers les modifications une par une, parfois il y aura un commettre Un qui est annulée ou modifiée par un plus tard commettre B dans le cadre de la même revue. Cela peut rendre l'examen plus difficile.

Si seulement l'un des développeurs a été de travailler sur le fichier pour la durée du changement, alors il est facile de faire un diff entre l'état d'origine du fichier et l'état final du fichier. Le problème se pose quand un autre développeur se fait sans rapport avec les changements dans le même fichier, ou même dans les mêmes fonctions.

Comment gérez-vous cette situation? Existe-il des outils qui, étant donné une séquence de patchs à un fichier, peut donner l'équivalent moral d'une diff entre la première et la dernière versions, mais seulement un sous-ensemble de ces patchs?

Venez pour penser à elle, j'ai pu créer un temporaire de la branche git à partir de avant la première liées à des changements, et de choisir les modifications pertinentes à l'examen. J'espère qu'il ne sera pas trop nombreux conflits qui doivent être résolus (et si il y a, alors l'ensemble du lot doivent être examinés à la fois de toute façon). D'autres idées?

Plus d'infos: Cela arrive à être un grand ancien système où un seul changement peut toucher plusieurs fichiers. Les fichiers sont gros et crufty - trop grand pour se contenter d'examiner le produit final, sans indication de ce qui pourrait avoir changé.

Prograide.com

Prograide est une communauté de développeurs qui cherche à élargir la connaissance de la programmation au-delà de l'anglais.
Pour cela nous avons les plus grands doutes résolus en français et vous pouvez aussi poser vos propres questions ou résoudre celles des autres.

Powered by:

X