Examiner une modification

Ce document explique comment examiner une modification. Tenez compte des critères suivants pour qu'une modification soit valide:

  • Ce changement s'inscrit-il dans 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 susceptibles de causer des problèmes à l'avenir ?
  • Ce changement suit-il les bonnes pratiques établies pour ce projet ?
  • Cette modification est-elle un bon moyen d'exécuter la fonction décrite ?
  • Ce changement engendre-t-il des risques de sécurité ou d'instabilité, ou affecte-t-il des applications existantes ?

Pour en savoir plus sur l'examen d'une modification, consultez la section Examiner une modification de la documentation de 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 et examiner cette modification, procédez comme suit:

  1. Cliquez sur le lien de l'e-mail pour effectuer la modification. L'écran Modifier les infos de Gerrit s'affiche. Cet écran fournit une description du changement, identifie le contributeur du changement et tous les examinateurs, et fournit une liste des fichiers associés au changement.

  2. Lisez la description de la modification pour comprendre son objectif.

  3. Cliquez sur un fichier pour examiner les modifications qui y sont apportées. Le fichier s'ouvre dans un écran de comparaison à deux volets, le fichier existant à gauche et le fichier contenant les modifications proposées à droite. Pour consulter le fichier:

    1. À l'aide de la souris, mettez en surbrillance (cliquez + faire glisser) la partie de code pour laquelle vous avez un commentaire ou une question. Le message "Appuyez sur c pour commenter" s'affiche.

    2. Appuyez sur c. Une zone de commentaires en mode brouillon s'affiche.

    3. Saisissez votre commentaire ou votre question, puis cliquez sur ENREGISTRER.

  4. Cliquez sur le numéro unique de la modification (en haut à gauche de l'écran) pour revenir à l'écran "Modifier les informations".

  5. Continuez à examiner chaque fichier de la modification.

  6. Lorsque vous avez terminé l'examen de tous les fichiers associés à la modification, revenez à l'écran "Modifier les informations", puis cliquez sur RÉPONDRE. L'écran d'envoi des votes s'affiche.

  7. Choisissez l'une des notes suivantes pour effectuer la modification:

    • J'ai l'impression que +1 me convient, mais quelqu'un d'autre doit également l'approuver.
    • 0 Aucun score

    • -1 Je préfère que ce contenu ne soit pas envoyé tel quel. Il y a des choses à corriger.

    • -2 Vous ne devez pas soumettre cette modification

  8. Cliquez sur 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 de les examiner. Sinon, vous avez terminé d'examiner la modification.