Clarify "Co-authored-by" usage in PRs
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 52/100
- Issue-Typ
- Dokumentation
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- github
- Bereich
- documentation
Rechercherichtung
Lies den vierten Absatz des Abschnitts „Handling Others' Code“ im devguide und sieh dir das verlinkte Discourse-Thema als Kontext an. Kläre, wann die Zeile Co-authored-by: von GitHub für wesentliche Beiträge durch Reviews oder vorgeschlagene Änderungen verwendet werden sollte, einschließlich ihres Zusammenhangs mit squash, automerge und direkten vorgeschlagenen Änderungen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 2.1k
- Forks
- 1k
- Ø Merge
- 2 T. 12 Std.
- Gemergte PRs (30 T.)
- 12
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus python/devguide
-
type-feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
type-feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
-
topic-building python type-feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
-
needs: decision topic-test type-bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
topic-dev process type-feature
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Alle Issues in python/devguide
Ähnliche Issues
-
🐛 Bug 🔔 Pending processing
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
jumpserver/jumpserver#17584 ·
-
link-check link-check:sphinx-theme
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
qgis/QGIS-Documentation#11275 ·
-
bug priority:normal ready-for-dev
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
OpenHands/extensions#626 · 1 Kommentar ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 90/100
CSCfi/sd-search-api#39 ·