Travail sur la revue de code
The 2008-11-28 at 13:11 by Loïc d'Anterroches filed under InDefero, bug tracking and more.
Voici une capture d’écran de ce que j’arrive à faire pour la revue du code, cette capture d’écran est réalisée par la soumission d’un patch contre une version donnée des sources :
À gauche, vous avez l’ancienne version du code et à droite la nouvelle. En comparaison avec le diff d’un commit, vous avez plus de lignes de contexte (point 1) car j’applique réellement le patch contre les sources, je peux donc extraire plus que les 3 lignes d’avant après d’un patch standard. Vous pouvez facilement visualiser la différence entre l’avant et l’après (point 2) et je fais la jonction entre les différents éléments du patch pour chaque fichier (point 3) car l’ajout d’un contexte de 10 lignes peut regrouper les éléments.
Les choses maintenant à ajouter correspondent à la revue elle-même. Cela va d’abord être un commentaire pour chaque fichier et un commentaire général. Ensuite, et ceci va nécessiter du javascript donc c’est moins important, l’ajout de commentaires ligne par ligne. Ce qui est bien c’est que je travaille avec un patch, donc, le patch peut avoir n’importe quelle origine, le diff d’un commit pour la revue post commit et le diff d’un contributeur pour la revue pré commit.
Comments from readers
Alexis Metaireau said:
Génial ! InDefero deviens vraiment de plus en plus intéressant. Merci pour ces billets qui permettent de bien nous tenir au courant des avancées :)