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.
Inhalt dieses Artikels
- Informationen zu Pull-Request-Reviews
- Unterhaltungen auflösen
- Erforderlicher Review
- Weiterführende Informationen
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. For more information, see "Requesting a pull request review."
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:
- If required reviews are enabled and a collaborator with write, admin, or owner access to the repository submits a review requesting changes, the pull request cannot be merged until the same collaborator submits another review approving the changes in the pull request.
- 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.
To indicate that a conversation on the Files changed tab is complete, click Resolve conversation.
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 administrators can require that all pull requests receive a specific number of approving reviews from people with write or admin permissions in the repository or from a designated code owner before they're merged into a protected branch. 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. However, you won't be able to merge your pull request until the required number of reviewers with write or admin permissions in the repository approve your pull request's changes in their review. 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“.