Die Gerrit Kochbuch Beispiel 8 verhindert nicht unbedingt der Autor seine / Bewertung ihre eigene Veränderung, sondern jemand anderes erfordert es bis +2 vor einreichen zu können.
Ich richte die Zugangskontrolle für mein Unternehmen in gerrit ein und in unserem aktuellen internen Prozess gibt es einen Cross-over zwischen Peer-Reviewern und Codern (sie sind in der Regel die gleiche Gruppe von Leuten). Wir möchten auch nur 1 Rezensenten benötigen, um den Code zu überprüfen und ihn einzureichen, wenn er gut aussieht.
Bei der Standardeinrichtung kann jeder Benutzer mit der Option +2: Looks good to me, approved seinen eigenen Code überprüfen.
Gibt es eine Möglichkeit, den Autor daran zu hindern, seinen eigenen Code zu überprüfen, aber dennoch den Code des anderen vollständig überprüfen zu können? Ich konnte keine Art von Ausschlussautor in den Setup- oder Berechtigungseinstellungen der Zugriffssteuerungsgruppe finden.
Die Gerrit Kochbuch Beispiel 8 verhindert nicht unbedingt der Autor seine / Bewertung ihre eigene Veränderung, sondern jemand anderes erfordert es bis +2 vor einreichen zu können.
Das funktioniert für mich, aber es ist ein schneller Hack:
Ich habe meine frühere Antwort optimiert, also nehme ich nicht an, dass Sie einen MySQL-Server verwenden.
Vielleicht möchten Sie die Logdatei irgendwo verschieben, wo sie einer normalen Log-Rotation unterliegt - vielleicht in ../ logs / comment-added.log.
%Vor%Ich habe versucht, die konfigurierbaren Bits in den Vordergrund zu ziehen. Rufen Sie diesen Dateikommentar-hook und auf setze es in $ gerrit_root / hooks, chmod es 755 oder ähnliches. Richten Sie einen Roboterbenutzer im Admin ein Gruppe, so kann der Haken die SQL-Schnittstelle (und kommentieren +2 auf Dinge mit genug + 1s).
Gerrit ermöglicht es Ihnen, prolog "send-Regeln" einzurichten, die definieren, wann eine Änderung übertragbar ist.
Die Dokumentation enthält mehrere Beispiele, einschließlich eines Beispiels, das verhindert, dass der Autor seine eigenen Änderungen genehmigt .
Ich habe gerade diesen Prolog-Filter für unsere Gerrit-Installation geschrieben. Ich habe es als ein submit_filter im übergeordneten Projekt getan, weil ich wollte, dass es für alle Projekte in unserem System gilt.
%Vor%Die Vorteile (IMO) dieser Regel gegenüber Regel # 8 aus dem Kochbuch sind:
Self-Reviewed wird nur angezeigt, wenn die Änderung blockiert wird, anstatt ein Non-Author-Code-Review -Label zu jeder Änderung reject(O) verwenden, bewirkt die Regel, dass das Label Self-Reviewed buchstäblich ein rotes Flag submit_filter anstelle von submit_rule wird diese Regel in einem übergeordneten Projekt installiert und gilt für alle Unterprojekte Bitte beachten Sie: Diese Regel wurde erstellt, um zu verhindern, dass Owner eine Änderung selbst überprüft, während das Beispiel aus dem Kochbuch mit dem Author verglichen wird. Abhängig von Ihrem Arbeitsablauf möchten Sie vielleicht die 2 gerrit:change_owner(O) Prädikate durch gerrit:commit_author(O) oder gerrit:commit_committer(O)
Sie können es über die GUI in der Registerkarte Zugriff tun. Gehe zu / refs / heads / section - & gt; Fügen Sie die Gruppe "Besitzer ändern" im Abschnitt "Label-Code-Review" hinzu - & gt; wähle -1 .. + 1
Dadurch erhält der Besitzer der Änderung das Recht, -1 bis +1
zu vergebenTags und Links gerrit