Ce document explique comment examiner une modification. Gardez à l'esprit les critères suivants pour qu'une modification soit 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 susceptibles de poser problème à l'avenir ?
- Cette modification respecte-t-elle les bonnes pratiques établies pour ce projet ?
- Cette modification est-elle un bon moyen d'effectuer la fonction décrite ?
- Ce changement présente-t-il des risques de sécurité ou d'instabilité, ou rend-il des applications existantes inutilisables ?
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. Pour examiner et vérifier la modification, procédez comme suit :
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 réviseurs, 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. Le fichier s'ouvre sur un écran de comparaison à deux volets, avec le fichier existant à gauche et le fichier avec les modifications proposées à droite. Pour examiner le fichier :
À l'aide de votre souris, mettez en surbrillance (cliquez et faites glisser) la partie du code sur laquelle vous avez un commentaire ou une question. Le message "Appuyez sur c pour commenter" s'affiche.
Appuyez sur c. Une zone de commentaire provisoire s'affiche.
Saisissez votre commentaire ou votre question, puis cliquez sur ENREGISTRER.
Cliquez sur le numéro unique de la modification (en haut à gauche de l'écran) pour revenir à l'écran "Infos sur la modification".
Continuez à examiner chaque fichier de la modification.
Une fois que vous avez examiné tous les fichiers de la modification, revenez à l'écran "Infos sur la modification" et cliquez sur RÉPONDRE. L'écran "Voter pour les exigences" s'affiche.
Choisissez l'un des scores suivants pour le changement :
- +1 Cela me semble bien, mais quelqu'un d'autre doit également approuver.
0 Aucun score
-1 Je préférerais que ce contenu ne soit pas envoyé tel quel, car il y a des choses à corriger.
-2 Cette modification ne doit pas être envoyée
Cliquez sur RÉPONDRE pour envoyer vos commentaires et votre note au développeur de la modification.
Si des problèmes doivent être résolus, le développeur les corrige et vous demande de les examiner. Sinon, vous avez terminé d'examiner la modification.