Gut prüfbare PRs
Die Gewohnheiten, die Ihren Teamkollegen beim Prüfen eines Pull Requests helfen, helfen auch HeyDeer.
Jeden Pull Request fokussiert halten
Ein Pull Request, der eine einzige Sache erledigt, erhält Befunde zu genau dieser Sache. Trennen Sie Refactorings, Umbenennungen und Dateiverschiebungen von Verhaltensänderungen, da sie sonst die tatsächlich geänderten Zeilen verdecken.
Absicht beschreiben und Issue verknüpfen
HeyDeer liest Titel und Beschreibung. Geben Sie an, wozu die Änderung dient, bei welchem Teil Sie am unsichersten sind und was Sie bewusst ausgelassen haben. Wenn der Autor Schreibzugriff hat, zählt ein genannter Kompromiss wie „Wiederholungsversuche sind nicht Teil dieses PR; verfolgt in #412“ als Entscheidung, ähnlich wie das Ablehnen eines Befunds.
Verknüpfen Sie auch das Issue. Wenn „Verknüpfte Issues lesen“ aktiviert ist, wie standardmäßig der Fall, kann HeyDeer darin nachlesen, welche Anforderungen die Änderung erfüllen soll. Siehe Was HeyDeer liest.
Generierte Dateien markieren
Markieren Sie generierten Code in .gitattributes mit linguist-generated, dann lässt HeyDeer ihn aus. HeyDeer liest das Attribut aus dem Basis-Branch, mergen Sie diese Änderung also zuerst in einem eigenen Pull Request.
Während der Iteration einen Entwurf öffnen
Automatische Prüfungen überspringen Entwürfe standardmäßig und berücksichtigen den Pull Request, sobald Sie ihn als „Ready for review“ markieren. Für frühes Feedback kommentieren Sie @heydeer review im Entwurf. Siehe Prüffilter.
Korrekturen als neue Commits pushen
Folgeprüfungen betrachten, was sich seit der letzten Prüfung geändert hat. Das funktioniert am besten, wenn Korrekturen als neue Commits eintreffen. Den Ziel-Branch hineinzumergen oder darauf zu rebasen ist unproblematisch. Pushen Sie zusammengehörige Korrekturen gemeinsam, um weniger automatische Prüfungen zu verbrauchen. Siehe Folgeprüfungen.
Auf abgelehnte Befunde antworten
Antworten Sie im Thread mit der Begründung, nicht nur mit „wird nicht behoben“, oder markieren Sie den Thread als erledigt. Eine Entscheidung durch jemanden mit Schreibzugriff teilt HeyDeer mit, das Problem nicht erneut anzusprechen. Siehe Rauschen reduzieren.