Skip to content

PR Checklist

Brug denne checklist inden du beder om review. Det sparer tid for alle.

Inden du åbner PR'en

Forfatter-checklist

Kode

  • Koden løser præcis det den skal — ikke mere
  • Ingen dead code, var_dump, dd() eller debug-output
  • Ingen hardcodede værdier der burde være config/env
  • declare(strict_types=1) i alle nye PHP-filer

Tests

  • Alle eksisterende tests er grønne lokalt
  • Ny funktionalitet er dækket af tests
  • Edge cases er testet (null, tom array, negative tal osv.)

Static analysis

  • vendor/bin/phpstan analyse src --level=8 er ren
  • vendor/bin/php-cs-fixer fix src/ --dry-run er ren

Dokumentation

  • CHANGELOG.md er opdateret (ved breaking changes eller ny feature)
  • Inline PHPDoc er opdateret hvis public API er ændret
  • README/handbook-docs er opdateret ved breaking changes

PR-beskrivelse

  • Titel følger Conventional Commits: feat: ..., fix: ... osv.
  • Beskrivelse forklarer hvorfor, ikke bare hvad
  • Breaking changes er tydeligt markeret
  • Linked til issue/ticket hvis relevant

Reviewer-guide

Som reviewer: godkend ikke en PR du ikke forstår. Stil spørgsmål — det er ikke din fejl at koden er svær at læse.

Fokusér på:

  • Korrekthed og edge cases
  • Sideeffekter og bagudkompatibilitet
  • Testdækning af kritisk logik
  • Naming og læsbarhed

Lad tooling (CS-Fixer, PHPStan, CI) håndtere formatering og typefejl — commenter ikke på whitespace.

Merge-regler

  • Minimum 1 approvende review fra et andet teammedlem
  • Alle CI-checks skal være grønne
  • Squash-merge til main (standard)
  • Branch slettes automatisk efter merge