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 ?
- Ce changement est-il valide dans l'architecture existante du projet ?
- Ce changement introduit-il des défauts de conception qui pourraient causer des problèmes dans à 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 ?
- Ce changement présente-t-il des risques de sécurité ou d'instabilité, ou endommage-t-il des applications existantes ?
Pour plus d'informations sur l'examen d'une modification, consultez la Examiner une modification de la documentation Gerrit.
Si le rôle d'examinateur d'une modification vous a été attribué, vous devriez recevoir un e-mail contenant un lien vers la modification. Pour examiner la modification :
Cliquez sur le lien de l'e-mail pour effectuer la modification. Infos sur la modification de Gerrit l'écran 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 du changement pour comprendre son objectif.
Cliquez sur un fichier pour examiner les modifications apportées. 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 consulter le fichier:
Utilisez la souris pour mettre en surbrillance (cliquez et faites glisser) la partie de code pour laquelle vous avez un commentaire ou une question. Le message "Appuyer 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 ENREGISTRER.
Cliquez sur le numéro unique de la modification (en haut à gauche de l'écran) pour revenir à l'écran "Modifier les informations".
Continuez à examiner chaque fichier de la modification.
Une fois que vous avez examiné tous les fichiers de la modification, revenez à l'écran "Informations sur la modification", puis cliquez sur RÉPONDRE. Votes pour la condition d'envoi s'affiche.
Choisissez l'une des notes suivantes pour effectuer la modification:
- +1 Cela me semble correct, mais une autre personne doit également approuver.
0 Aucun score
-1 Je préfère que cette demande ne soit pas envoyée telle quelle. Il y a des choses à corriger.
-2 Vous ne devez pas soumettre cette modification
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.