Improve PR review checklist

Ouverte
#962 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-2 jours
Accessibilité débutants
32/100
Type d'issue
Documentation
Clarté
Plutôt claire
Activité
À l'abandon
Stack technique
github
Domaine
documentation

Piste de recherche

Lisez les sections « Reviewing » des pages pull-request-lifecycle et triaging, puis examinez la checklist du commentaire lié de la pull request CPython. Regroupez la checklist pertinente dans pull-request-lifecycle et supprimez les indications de Reviewing en double de triaging ; le travail est terminé lorsque les deux pages ne répètent plus les mêmes informations.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

topic-pull requests topic-triage type-bug type-feature

In https://github.com/python/cpython/pull/93823#issuecomment-1156152572 @erlend-aasland wrote a checklist about reviewing PRs.

In https://github.com/python/devguide/pull/914#issuecomment-1207257987, I suggested:

SGTM -- that's a nice list. I think it would be easier to add it in a separate PR though.
Reviewing PRs is currently documented in two places:

The former has been likely written from scratch after the HG->Git migration, whereas the latter formerly documented how to review patches attached to a bpo issue and then it has been adapted to GitHub. I think the best approach would be to enhance the pull-request-lifecycle with your list, and remove the duplicated info in triaging.

Langage dominant
Python
Étoiles
2.1k
Forks
1k
Merge moyen
2 j 12 h
PR mergées (30 j)
12

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de python/devguide

Toutes les issues de python/devguide

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.