Wie kann ich Peer Reviews zu GitHub-Pull-Anfragen durchführen?

12

Wir wechseln von Bitbucket zu GitHub und haben Probleme mit Peer-Code-Überprüfungen, die bei Bitbucket wie folgt reibungslos funktionierten:

  1. Autor hat eine Pull-Anfrage geöffnet (GitHub: das gleiche)
  2. Der Autor hat seine / ihre Kollegen als Rezensenten hinzugefügt (GitHub: Kämpfe hier mit mehreren Bearbeitern)
  3. Prüfer entweder:
    1. PR mit grünem Häkchen bestätigt (GitHub: ??)
    2. Kommentare hinzugefügt (GitHub: das gleiche)
    3. Erstellte leichte Aufgaben (GitHub: ähnlich, wenn - [ ]in der PR-Beschreibung Syntax verwendet wird; schade, dass es für Aufgaben nicht funktioniert)
  4. Es gibt eine Liste von PRs, bei denen ich auf einen Blick sehen kann, welche überprüft und zum Zusammenführen bereit sind und welche weitere Aufmerksamkeit benötigen (GitHub: ??)

Ich möchte darauf hinweisen, dass wir Code-Überprüfungstools von Drittanbietern nach Möglichkeit vermeiden und mit einigen Problemumgehungen auf Vanilla GitHub bleiben möchten.

code-reviews github pull-requests Borek Bernard
quelle
1
Klingt so, als ob Sie vorzeitig wechseln würden. Warum sollten Sie trotzdem wechseln, besonders wenn das neue Ding nicht alle Funktionen bietet, die Sie benötigen?
Kindermädchen
Schreiben Sie einen Kommentar zu Ihrer Anfrage und markieren Sie @, wann immer Sie eine Benachrichtigung erhalten möchten. Der Rezensent kann Tags hinzufügen, um seine Rezensionsmeinung anzuzeigen.
Wilbert
Beachten Sie auch, dass Sie ein ganzes Team @mention
Whymarrh

Antworten:

6

Nach dem, was ich gesehen habe, werden die meisten dieser Schritte auf Github nach Konvention und nicht nach einem offiziellen, von Github bereitgestellten Verfahren ausgeführt.

Mein Arbeitgeber nutzt Github, ich führe eine ganze Reihe kleiner Open-Source-Projekte und leiste gelegentlich Beiträge zu anderen Open-Source-Projekten.

So habe ich es normalerweise gesehen:

Autor, der seine Kollegen als Reviewer hinzufügt:

Dies ist von Projekt zu Projekt unterschiedlich, aber im Allgemeinen tragen die zugewiesenen Peer-Reviewer alle zum Projekt bei .

Open-Source-Projekte scheinen eine grobe Hierarchie zu haben - möglicherweise besteht ihre Konvention darin, sie erst dann zusammenzuführen, wenn ein "Kern" -Beitragender das Okay gegeben hat.

In dem Geschäft, in dem ich derzeit beschäftigt bin, schließen wir uns zusammen, nachdem einer der rund ein halbes Dutzend Entwickler im Team seine Zustimmung erteilt hat.

In seltenen Fällen kann jemand im Team einen Kommentar verwenden, um einen anderen Entwickler speziell darauf hinzuweisen, dass er den Code einer Peer-Überprüfung unterziehen sollte, bevor er zusammengeführt wird. Andernfalls kann jedoch jeder, der als erster dort ankommt und dies wünscht, eine Überprüfung durchführen und Kommentare abgeben.

Reviewer Zustimmung:

Die Genehmigung wird normalerweise durch einen Kommentar auf der Pull-Anfrage angezeigt, der "+1" oder "lgtm" lautet (sieht für mich gut aus).

Leichte Aufgaben:

Ich habe auch die Kontrollkästchen verwendet, aber in den meisten Fällen wird jeder Kommentar zu einer Pull-Anfrage als implizite "Aufgabe" betrachtet, die entweder durch Folgendes gelöst wird:

  • Ändern des Codes, den die Zeile kommentiert
  • mit einem anderen Kommentar antworten

Sehen Sie auf einen Blick, was genehmigt wurde und was noch überprüft werden muss:

Ich habe die Erweiterung " Looks Good To Me" für Chrome verwendet, mit der Sie eine solche Ansicht vom Bildschirm "Pull Requests" aus erhalten. Die Ansicht der Pull-Requests-Liste scheint jedoch durch die letzten Github-Änderungen beschädigt worden zu sein.

TehShrike
quelle