Ce document explique comment examiner une modification. Gardez à l'esprit les critères suivants pour une modification valide :
- Cette modification correspond-elle à l'objectif déclaré de ce projet ?
- Cette modification est-elle valide dans l'architecture existante du projet ?
- Cette modification introduit-elle des défauts de conception qui pourraient causer des problèmes à l'avenir ?
- Cette modification respecte-t-elle les bonnes pratiques établies pour ce projet ?
- Cette modification est-elle un bon moyen d'exécuter la fonction décrite ?
- Cette modification introduit-elle des risques de sécurité ou d'instabilité, ou interrompt-elle des applications existantes ?
Pour en savoir plus sur l'examen d'une modification, consultez la section Examiner une modification de la documentation Gerrit.
Si vous êtes désigné comme examinateur d'une modification, vous devriez recevoir un e-mail contenant un lien vers la modification. Suivez ces étapes pour examiner la modification :
Cliquez sur le lien de l'e-mail vers la modification. L'écran "Change Info" (Informations sur la modification) de Gerrit s'affiche. Cet écran fournit une description de la modification, identifie le contributeur de la modification et tous les examinateurs, et fournit une liste des fichiers associés à la modification.
Lisez la description de la modification pour comprendre son objectif.
Cliquez sur un fichier pour examiner les modifications apportées à celui-ci. Le fichier s'ouvre dans un écran de comparaison à deux volets, avec le fichier existant à gauche et le fichier avec les modifications proposées à droite. Pour examiner le fichier :
Utilisez votre souris pour mettre en surbrillance (cliquez et faites glisser) la partie du code pour laquelle vous avez un commentaire ou une question. Le message "Press c to comment" (Appuyez sur c pour commenter) s'affiche.
Appuyez sur c. Une zone de commentaire brouillon s'affiche.
Saisissez votre commentaire ou votre question, puis cliquez sur SAVE (ENREGISTRER).
Cliquez sur le numéro unique de la modification (en haut à gauche de l'écran) pour revenir à l'écran "Change Info" (Informations sur la modification).
Continuez à examiner chaque fichier de la modification.
Lorsque vous avez terminé d'examiner tous les fichiers de la modification, revenez à l'écran "Change Info" (Informations sur la modification) et cliquez sur REPLY (RÉPONDRE). L'écran "Submit requirement votes" (Envoyer les votes sur les exigences) s'affiche.
Choisissez l'un des scores suivants pour la modification :
- +1 Cela me convient, mais quelqu'un d'autre doit également approuver.
0 Aucun score
-1 Je préférerais que cela ne soit pas envoyé tel quel, il y a des choses à corriger.
-2 Cette modification ne doit pas être envoyée.
Cliquez sur REPLY (RÉPONDRE) pour envoyer vos commentaires et votre score au développeur de la modification.
S'il y a des problèmes à résoudre, le développeur les corrige et vous demande d'examiner les corrections. Sinon, vous avez terminé d'examiner la modification.