Was ist bei GitHub der Unterschied zwischen Prüfer und Beauftragter?

185

Eine am 7. Dezember 2016 im GitHub-Blog angekündigte Funktion führte die Option ein , einer Pull-Anfrage Prüfer hinzuzufügen

GitHub-Überprüfungsoption

Sie können jetzt explizit eine Überprüfung von Mitarbeitern anfordern, um leichter anzugeben, wer Ihre Pull-Anforderung überprüfen möchte.

Sie können auch eine Liste der Personen, von denen Sie auf eine Überprüfung warten, in der Seitenleiste der Pull-Anforderungsseite sowie den Status der Bewertungen von Personen anzeigen, die sie bereits verlassen haben.

Das explizite Festlegen eines Prüfers für eine PR erfolgte jedoch bereits durch Zuweisen von Personen ( Option " Empfänger ").

Welche Rolle spielt jede Option, da beide Optionen jetzt verfügbar sind, da beide dasselbe Endziel haben?

Cezar Augusto
quelle
1
Wann wird "Assignee Feature" zum ersten Mal veröffentlicht? Gibt es einen Artikel, der es einführt?
Babeyh

Antworten:

133

BEARBEITEN:

Mit mehreren OSS - Maintainer Nach der Diskussion, Rezensenten ist definiert als das, was soll das Wort sein: zu der Bewertung (jemandes Code) und „Zessionar“ hat eine lockerere definiton unten erläutert.

Für "Prüfer" : Jemand, den Sie den Code überprüfen möchten. Nicht unbedingt die Person, die für diesen Bereich verantwortlich ist oder für die Zusammenführung des Commits verantwortlich ist. Kann jemand sein, der zuvor an diesem Codeabschnitt gearbeitet hat, wie GitHub automatisch vorschlägt.

Für "Beauftragter" : Bis zum Team / Betreuer des Projekts, was es bedeutet, und es gibt keine strenge Definition. Dies kann der PR-Eröffner oder eine für diesen Bereich verantwortliche Person sein (die die PR nach Abschluss der Überprüfung akzeptiert oder sie einfach schließt). Es liegt nicht an GitHub zu definieren, was es für Projektbetreuer offen lässt, was am besten zu ihrem Projekt passt.

Vorherige Antwort:

Ok, ich werde meine eigene Frage beantworten.

Für die PR von Benutzern mit Schreibzugriff: Der Beauftragte ist dieselbe Person, die die PR geöffnet hat, und der Prüfer ersetzt die alte Funktion des Empfängers (Überprüfungscode), da dies eine Person der Wahl des Empfängers ist.

Für PR von Benutzern ohne Schreibzugriff (externe Mitwirkende): Jemand mit Schreibzugriff würde sich selbst (oder ein anderes Mitglied mit Schreibberechtigung) damit beauftragen, die PR (Reviewer) zu überprüfen. Der Empfänger ist leer.

Für unvollendete PR von externen Mitwirkenden : Das Mitglied mit Schreibzugriff würde die unvollendete Arbeit übernehmen und für sie zuweisen. Sie wird als Beauftragte für die Beendigung der Aufgabe verantwortlich sein . Da der Hauptgrund für PRs die Überprüfung von Änderungen ist, würde sie einige andere Personen auswählen, um die Änderungen zu überprüfen.

Cezar Augusto
quelle
24
Für jedes neue Teammitglied sollte ich einen Link zu dieser Antwort senden, um zu erklären, wie man mit Beauftragten und Gutachtern umgeht. Was mich zu dem Gedanken führt, dass hier etwas grundlegend falsch ist :(
Andrey Kuleshov
Muss ein Empfänger Schreibzugriff haben?
Emre Sülün
Gibt es einen Unterschied im Verhalten der E-Mail-Benachrichtigung zwischen den beiden?
jxramos
26

In GitHub ist ein Prüfer eine Person, die die Pull-Anfrage prüft. Ein Projektbesitzer kann eine Überprüfung von jedem der Betreuer anfordern. Er kann sogar eine Option festlegen, sodass die Pull-Anforderung nur zusammengeführt werden kann, wenn sie von einem der Betreuer mit Schreibzugriff überprüft wird.

Laut offizieller Github-Dokumentation ist der Beauftragte eine Person, die an bestimmten Problemen arbeitet und Anfragen abruft. Es wird manchmal als Rezensent verwirrt. Es ist eigentlich dazu gedacht, bei Problemen verwendet zu werden, anstatt eine Anfrage zu ziehen, damit wir, wenn wir ein Problem erhalten, jemanden beauftragen können, es zu beheben. In einer Pull-Anfrage bezieht sich ein Beauftragter auf eine Person, die für das Zusammenführen dieser Pull-Anfrage verantwortlich ist, nachdem sie Kommentare und Änderungsanforderungen von anderen Betreuern erhalten hat.

Gautam Krishna R.
quelle
2
Vielen Dank für die Antwort, aber ich denke nicht, dass sie die Frage vollständig beantwortet. Sie können jemandem ein Problem zuweisen (sie wird also die Problemempfängerin sein), aber wenn die PR gesendet wird, ist jemand der Prüfer (PR-Beauftragte), und zu diesem Zeitpunkt ist mir der Unterschied zwischen Beauftragter und noch nicht klar Rezensent.
Cezar Augusto
14

Gemäß akzeptierter Antwort. Ja, "Beauftragter" hat eine lockerere Definition und kann je nach Bedarf eines Teams unterschiedlich verwendet werden.

In unserem Team von 8 Entwicklern haben wir in den meisten PRs 1 Prüfer, der Änderungen vorschlägt und letztendlich die PR genehmigt. Während der Überprüfungsphase ist "Beauftragter" die Person, die die PR geöffnet hat. Wenn PR später von einem anderen Entwickler übernommen wird, wird ein neuer "Beauftragter" hinzugefügt. Sobald PR genehmigt und für die Qualitätssicherung oder direkte Zusammenführung bereit ist, wird ein neuer Qualitätssicherungsempfänger hinzugefügt. Auf diese Weise wächst die Liste der "Empfänger".

Wir verwenden "Rechtsnachfolger", um folgende Personen gemeinsam zu bezeichnen:

  1. Pull Request Author
  2. Autor, der an PR-Änderungsvorschlägen arbeitet (normalerweise wie 1)
  3. QA Person beteiligt
  4. Verantwortliche Person für die Zusammenführung (normalerweise wie 2 oder 3)

Die Verwendung von "Beauftragter" erleichtert das Auffinden der PR in Zukunft. Eines meiner Projekte hat> 3000 PRs.

is:open is:pr author:raya-dumas

is:closed is:pr assignee:raya-dumas

Oder nur author:raya-dumasum alle vom Autor erstellten Elemente zu finden (Probleme, PRs)

und andere ähnliche Abfragen, um den Suchvorgang zu vereinfachen. "Meilensteine" sind ebenfalls sehr hilfreich, um die PR-Suche zu vereinfachen.

Screenshot Github, 4. Quartal 2017

lal
quelle
Sehr gut erklärt.
Nitin Gaur
Es sollte erwähnt werden, dass Sie nur nach dem Autor suchen können: my-github-handle, um herauszufinden, welche PR eine Person erstellt hat
Wisienkas
1

Vorher hatte GitHub nur ein Empfängerfeld und kein Prüferfeld. Damals gab es keine Unterscheidung, daher wurde das Feld des Empfängers am häufigsten als Prüferfeld verwendet.

Verwenden Sie sie jedoch so, wie es zu Ihrem Projekt passt.

Ericcurtin
quelle