PR Richtlinien
Richtlinien, die Prüfer und Beitragende folgen sollten, wenn sie Pull Requests öffnen und überprüfen.
Pull Request Richtlinien
Um einen reibungslosen und effizienten Überprüfungsprozess zu gewährleisten, sollten sowohl Beitragende als auch Überprüfer die folgenden Richtlinien einhalten, wenn Pull-Requests geöffnet und überprüft werden.
Für Beitragende
- Beschreibende Titel und Beschreibungen
- Verwende klare und prägnante Title, die die Änderungen zusammenfassen.
- Gib detaillierte Beschreibungen darüber, was deine PR macht, warum die Änderungen notwendig sind und sonstigen relevanten Kontext.
- Link zu Issues
- Füge
GH-ISSUE-NUMBERzu deinem Titel hinzu um auf das PR-relevante Issue zu verweisen. Dies ist für alle PRs erforderlich.
- Füge
- Kleine, fokussierte Änderungen
- Ziel ist es, die PRs klein zu halten und sich auf ein einzelnes Problem oder Feature zu konzentrieren. Dies macht es für die Prüfer einfacher zu verstehen und Feedback zu geben.
- Codequalität
- Stelle sicher, dass dein Code den Projektstandards und Best Practices folgt.
- Füge gegebenenfalls Kommentare hinzu, um komplexere Logik zu erklären.
- Testen
- Teste deine Änderungen gründlich, bevor du eine PR einreichst.
- Falls zutreffend, füge Komponententests oder Integrationstests hinzu, um Änderungen zu überprüfen.
- Dokumentation
- Wenn nötig, aktualisiere oder füge Dokumentation hinzu, um deine Änderungen wiederzuspiegeln.
- Commitnachrichten
- Verwende klare und beschreibende Commitnachrichten, die dem conventional commits Format folgen.
- Konflikte überprüfen
- Bevor du deine PR einrichst, stelle sicher, dass deine Branch auf dem neuesten Stand der Basis-Branch ist um Merge-Konflikte zu minimieren.
Für Prüfer
- Du sollst PRs gründlich überprüfen und gegebenenfalls Änderungen anfordern.
- Gib konstruktives Feedback und Verbesserungsvorschläge.
- Stelle sicher, dass die Änderungen den Codestandards und Best Practices folgen.
- Stelle die PR mithilfe der Dokploy-Integration bereit, um sicherzustellen, dass die Änderungen wie erwartet funktionieren.
- Stelle sicher, dass die Dokumentation bei Bedarf aktualisiert wurde.
- Bestätige die PR nur, wenn du mit den Änderunge zufrieden bist und alle Tests funktioniert haben.