Informationen zu Pull-Request-Reviews
Mit Reviews können Mitarbeiter die in Pull Requests vorgeschlagenen Änderungen kommentieren, die Änderungen genehmigen oder weitere Änderungen anfordern, bevor der Pull Request gemergt wird. Repository-Administratoren können festlegen, dass alle Pull Requests vor dem Mergen genehmigt werden müssen.
In diesem Artikel:
Informationen zu Pull-Request-Reviews
Nachdem ein Pull Request geöffnet wurde, kann jeder mit Lesezugriff die von ihm vorgeschlagenen Änderungen überprüfen und kommentieren. Weitere Informationen finden Sie unter „Vorgeschlagene Änderungen in einem Pull Request überprüfen“.
Repository-Inhaber und Mitarbeiter können von einer bestimmten Person einen Review des Pull Requests anfordern. Organisationsmitglieder können auch einen Review eines Pull Requests von einem Team mit Lesezugriff auf das Repository anfordern. Weitere Informationen finden Sie unter „Einen Pull-Request-Review anfordern“.
Reviews ermöglichen eine Diskussion der vorgeschlagenen Änderungen und tragen dazu bei, dass die Änderungen den Beitragsrichtlinien des Repositorys und anderen Qualitätsstandards entsprechen. Sie können definieren, welche Personen oder Teams bestimmte Codetypen oder -bereiche in einer CODEINHABER-Datei besitzen. Wenn durch einen Pull Request Code mit einem definierten Inhaber geändert wird, wird diese Person oder dieses Team automatisch als Reviewer angefordert. Weitere Informationen finden Sie unter „Informationen zu Codeinhabern“.
Ein Review kann drei verschiedene Status haben:
- Comment (Kommentieren): Senden Sie allgemeines Feedback, ohne die Änderungen ausdrücklich zu genehmigen oder zusätzliche Änderungen anzufordern.
- Approve (Genehmigen): Senden Sie Feedback und genehmigen Sie das Mergen der im Pull Request vorgeschlagenen Änderungen.
- Request changes (Änderungen anfordern): Fordern Sie mit Ihrem Feedback Änderungen an, die vor dem Mergen des Pull Requests vorgenommen werden müssen.
Tipps:
- Wenn erforderliche Reviews aktiviert sind und ein Mitarbeiter mit dem Zugriff write (Schreiben), admin (Administrator) oder owner (Inhaber) auf das Repository einen Review zum Anfordern von Änderungen absendet, kann der Pull Request erst gemergt werden, wenn derselbe Mitarbeiter einen anderen Review absendet, der die Änderungen in dem Pull Request genehmigt.
- Repository-Inhaber und Administratoren können einen Pull Request sogar dann mergen, wenn er keinem genehmigenden Review unterzogen wurde oder wenn ein Reviewer, der die Änderungen angefordert hat, die Organisation verlassen hat oder nicht verfügbar ist.
- Wenn erforderliche Reviews und das Verwerfen alter Reviews aktiviert sind und ein den Code ändernder Commit an den Branch eines genehmigten Pull Requests übertragen wird, wird die Genehmigung verworfen. Der Pull Request muss erneut überprüft und genehmigt werden, bevor er gemergt werden kann.
- Wenn verschiedene offene Pull Requests jeweils einen Head-Branch aufweisen, der auf denselben Commit verweist, können Sie sie nicht mergen, wenn der Review bei einer oder bei beiden ausstehend ist oder abgelehnt wurde.
Sie können alle Reviews eines Pull Requests in der Zeitleiste der Unterhaltung anzeigen, und Sie können Reviews von Repository-Inhabern und Mitarbeitern im Merge-Feld des Pull Requests sehen.
Tip: You can find a pull request where you or a team you're a member of is requested for review with the search qualifier review-requested:[USERNAME] or team-review-requested:[TEAMNAME]. For more information, see "Searching issues and pull requests."
Unterhaltungen auflösen
Sie können eine Unterhaltung in einem Pull Request auflösen, wenn Sie den Pull Request geöffnet haben oder Sie über Schreibzugriff auf das Repository verfügen, in dem der Pull Request geöffnet wurde.
Klicken Sie auf Resolve conversation (Unterhaltung auflösen), um anzugeben, dass eine Unterhaltung auf der Registerkarte Files changed (Dateien geändert) abgeschlossen ist.
Die gesamte Unterhaltung wird reduziert und als aufgelöst markiert. Dadurch lassen sich Unterhaltungen einfacher auffinden, die noch angegangen werden müssen.
Erforderlicher Review
Repository-Administratoren können vorschreiben, dass alle Pull Requests eine bestimmte Anzahl von genehmigenden Reviews von Personen mit den Berechtigungen write (Schreiben) oder admin (Administrator) im Repository oder vom designierten Codeinhaber erhalten müssen, bevor sie in einen geschützten Branch gemergt werden. Weitere Informationen finden Sie unter „Informationen zu geschützten Branches“.
Wenn erforderliche Reviews aktiviert sind, kann jeder mit Zugriff auf das Repository Änderungen in einem Pull Request genehmigen. Sie können Ihren Pull Request jedoch erst dann mergen, wenn die erforderliche Anzahl an Reviewern mit den Berechtigungen write (Schreiben) oder admin (Administrator) im Repository die Änderungen Ihres Pull Requests im jeweiligen Review genehmigt hat. Weitere Informationen zu Repository-Berechtigungsebenen finden Sie unter „Berechtigungsebenen für die Repositorys einer Organisation“. Wenn ein Review durch einen designierten Codeinhaber erforderlich ist und sich der Pull Request auf den Code auswirkt, der einen designierten Inhaber aufweist, ist eine Genehmigung dieses Inhabers erforderlich.
Weitere Informationen finden Sie unter „Informationen zu erforderlichen Reviews für Pull Requests“.
Tipp: Bei Bedarf können Personen mit Administratorzugriff oder Schreibzugriff auf ein Repository einen Pull-Request-Review ablehnen. Weitere Informationen finden Sie unter „Einen Pull-Request-Review ablehnen“.