Clarify "Co-authored-by" usage in PRs
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 1/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 52/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- github
- Domaine
- documentation
Piste de recherche
Lisez le quatrième paragraphe de la section « Handling Others' Code » du devguide et consultez le sujet Discourse lié pour avoir le contexte. Clarifiez quand la ligne Co-authored-by: de GitHub doit être utilisée pour des contributions importantes sous forme de revue ou de modifications suggérées, notamment en ce qui concerne squash, automerge et les modifications suggérées directement.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
As brought up in a Discourse committers topic, the devguide could be more clear about Pull Request attribution on GitHub.
Specifically, in the fourth paragraph of "Handling Others' Code", the Co-authored-by: feature of GitHub is mentioned, but the use case of it is not defined.
Since GitHub will automatically attribute the primary author of the PR when using "squash and merge" or the "automerge", the Co-authored-by: is useful to add just prior to merging for anyone that significantly contributed to the PR, either via review or direct suggested changes. When using the GitHub UI to directly commit a suggested change to the PR, a Co-authored-by: line is automatically added to the commit message. Otherwise though, it has to be done manually for people other than the author to be attributed for contributing to the PR.
With this being a rather substantial part of the workflow for GitHub PRs, it seems to be worthwhile to mention when Co-authored-by: should be used rather than mentioning the feature without elaborating further.
- 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
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de python/devguide
-
type-feature
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
type-feature
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
topic-building python type-feature
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
-
needs: decision topic-test type-bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
topic-dev process type-feature
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Toutes les issues de python/devguide
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
-
🐛 Bug 🔔 Pending processing
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
jumpserver/jumpserver#17584 ·