Fournir une révision de demande d'extraction

Tout le monde est invité à examiner toute contribution au projet BeeWare. Il y a quelques considérations importantes à prendre en compte avant de commencer.

Réfléchissez avant de réviser

Avant de vous engager dans une évaluation, RÉFLÉCHISSEZ. En tant qu'évaluateurs, nous devons nous demander si la réponse que nous nous apprêtons à envoyer est.. :

• C'est vrai. Il faut toujours s'efforcer de fournir des suggestions et des informations exactes.
• Utile. Nous fournissons des conseils sur la manière d'améliorer la soumission ; ces conseils doivent clairement identifier la source d'un problème ou d'un cas d'utilisation non pris en compte, et idéalement fournir une voie à suivre pour résoudre ou satisfaire le problème.
• Inspirer. C'est à nous d'inspirer l'auteur pour qu'il ait envie de travailler sur les changements que nous avons demandés.
• Nécessaire. On s'attend à ce que l'auteur lise tout ce que nous publions ; nous devons respecter son temps et ses efforts en ne publiant que lorsque c'est nécessaire.
• Gentillesse. Il existe de multiples façons de présenter le même retour d'information ; nous devons nous assurer que nous choisissons d'être aimables, encourageants et constructifs dans nos propos.

Il est tout à fait possible de PENSER, tout en fournissant une évaluation efficace. Les concepts évoqués ci-dessus n'empêchent pas de signaler les problèmes que l'on peut rencontrer dans un PR. Les contributeurs n'auront pas la possibilité d'améliorer leur contribution s'ils ne sont pas conscients des points à améliorer. L'important est de rester conscient de la manière dont vous présentez ce retour d'information. Essayez de dépersonnaliser votre évaluation. Au lieu de dire "Vous avez fait une erreur", vous pouvez dire "Ce code pourrait être amélioré". Examinez le code, pas l'auteur.

Il est important de ne pas oublier de fournir un retour d'information positif en plus d'identifier les domaines à améliorer. Si, par exemple, les modifications sont particulièrement utiles, si vous faites quelque chose de particulièrement intelligent ou si vous découvrez une API que vous ne connaissiez pas, faites-le savoir à l'auteur ! Ne sous-estimez jamais l'effet que peut avoir le fait de souligner quelque chose que quelqu'un a fait correctement ou bien, au milieu d'une situation où tout ce que vous avez souligné est un problème qui doit être résolu.

Suggestions de révision sur GitHub

L'interface de révision de GitHub dispose d'un mécanisme pour les suggestions de changement, dans lequel vous pouvez fournir le changement exact que vous suggérez en remplacement du contenu existant. Gardez à l'esprit que, jusqu'à ce qu'ils soient acceptés et livrés, ces changements suggérés ne seront pas exécutés par la pré-commission et les vérifications de linting. Par conséquent, cette fonctionnalité devrait être utilisée pour des changements plus petits, car plus le changement suggéré est important, plus il est susceptible d'introduire des problèmes.