heydeer Anmelden
Best Practices

Teams mit menschlichen Prüfern

Wenn Menschen jeden Pull Request prüfen, fließt viel ihrer Zeit in Details und Randfälle. HeyDeer übernimmt diese bei jedem Pull Request, damit sich deine Prüfer auf Design- und Produktentscheidungen konzentrieren können.

Was HeyDeer übernimmt

  • Details und Randfälle. HeyDeer prüft bei jedem Pull Request Fehlerbehandlung, Grenzwerte, Nebenläufigkeit und die Aufrufer, die eine Änderung beschädigt.
  • Entlastung für Prüfer. Prüfer müssen nicht mehr jede Zeile nach Fehlern absuchen und haben so Zeit und Aufmerksamkeit für die Entscheidungen, die nur sie treffen können.
  • Genau im Detail. Bei 1.000 internen Pull Requests hat HeyDeer die Korrektheit im Detail genauer geprüft als menschliche Prüfer.

Empfohlenes Setup

Ein Pull Request wird geöffnet
HeyDeer prüft die Details
Der Autor behebt die Befunde
Ein Prüfer bewertet das Design und mergt
  1. Lass jeden Push prüfen. Setze unter Arbeitsbereichseinstellungen → Prüfregeln „Prüfzeitpunkt“ auf „Bei jedem Push“. Siehe Automatische Prüfungen.
  2. Lass den Check bei Problemen fehlschlagen. Aktiviere unter „Ergebnisse veröffentlichen“ die Option „Check bei gefundenen Problemen fehlschlagen lassen“. Siehe Der Check HeyDeer Review.
  3. Mach den Check in GitHub zur Pflicht. Aktiviere in den Branch-Regeln für main „Require status checks to pass before merging“ und füge HeyDeer Review hinzu. Ein Pull Request kann dann erst gemergt werden, wenn seine Befunde behoben oder abgelehnt sind.
  4. Lass die menschliche Prüfung auf dem Ergebnis von HeyDeer aufbauen. Bitte deine Prüfer, einen Pull Request erst zu öffnen, wenn sein Check bestanden ist, und ihre Zeit auf Design, Produktverhalten und alles zu verwenden, was HeyDeer für die Entscheidung einer Person markiert.
Einige Checks waren nicht erfolgreich
HeyDeer Review — Die Prüfung hat Probleme gefunden, die bearbeitet werden müssen
CI / test — Erfolgreich
Mergen ist blockiert
Ist der Check erforderlich, kann ein Pull Request erst gemergt werden, wenn seine Befunde behoben oder abgelehnt sind.

Optional: Routineänderungen automatisch genehmigen

Aktiviere Automatische Genehmigung und belasse „Änderungen mit erforderlicher menschlicher Genehmigung“ auf „Wesentliche Architekturänderungen und Änderungen an zentralem Verhalten“. HeyDeer genehmigt dann routinemäßige Pull Requests, sobald keine Probleme mehr offen sind, und überlässt wesentliche Änderungen einer Person; dabei nennt es die Entscheidung, die sie treffen muss.

Ist in deinen Branch-Regeln genau eine Genehmigung erforderlich, können routinemäßige Pull Requests gemergt werden, ohne auf einen Prüfer zu warten. Aktiviere „Dismiss stale pull request approvals when new commits are pushed“, damit jede Genehmigung den neuesten Code abdeckt.

Standards als Prüfungen festhalten

Eine Regel, die deine Prüfer in Kommentaren immer wieder anmerken, gehört in eine Prüfung. Lege für jede Regel eine Markdown-Datei in .agents/checks/ an, und HeyDeer wendet sie auf jeden Pull Request an, der passende Dateien ändert. Siehe Repository-Prüfungen.

.agents/checks/migrations.md

---
files: "migrations/**/*.sql"
---
The previous release keeps running while a migration
deploys. Report a migration that:

- drops or renames a table or column that code on the
  base branch still uses;
- adds a NOT NULL column without a default;
- creates an index on an existing table without
  CONCURRENTLY.