pull_request_template.md

📝 Beschreibung

🎯 Ziel

🔗 Verknüpfte Issues

Closes # Related to #

🔄 Art der Änderung

• 🐛 Bug Fix (non-breaking change)
• ✨ New Feature (non-breaking change)
• 💥 Breaking Change (fix oder feature mit breaking changes)
• 📝 Dokumentation
• 🔧 Refactoring
• ⚡ Performance Improvement
• 🧪 Tests
• 🏗️ Infrastructure/CI/CD

🧪 Testing

Wie wurde getestet?

• Unit Tests
• Integration Tests
• Manual Testing
• E2E Tests

Test-Abdeckung

• Neue Tests für neue Features/Bugfixes
• Bestehende Tests aktualisiert
• Alle Tests bestehen lokal

📋 Checkliste

Code Quality

• Code folgt den Projekt-Konventionen
• Self-Review durchgeführt
• Code-Kommentare in komplexen Bereichen
• Keine Debug-Ausgaben oder TODO-Kommentare
• Keine Compiler-Warnungen

Dokumentation

• README/Docs aktualisiert (falls nötig)
• API-Dokumentation aktualisiert
• Kommentare und Docstrings vorhanden
• CHANGELOG aktualisiert (falls verwendet)

Dependencies

• Keine neuen Dependencies ohne Grund
• Dependencies auf Sicherheit geprüft
• package.json/requirements.txt aktualisiert

Security

• Keine Secrets/Credentials im Code
• Input-Validierung implementiert
• Security Best Practices befolgt
• Keine bekannten Vulnerabilities

Breaking Changes

• Breaking Changes sind dokumentiert
• Migration Guide bereitgestellt
• Betroffene Teams informiert

📸 Screenshots / Videos

Vorher

Nachher

🚀 Deployment Notes

• Datenbank-Migrationen erforderlich
• Config-Änderungen erforderlich
• Environment Variables hinzugefügt/geändert
• Cache-Clear notwendig

📊 Performance Impact

• Performance-Tests durchgeführt
• Keine negativen Performance-Auswirkungen
• Performance-Verbesserungen dokumentiert

🔍 Review Focus

ℹ️ Zusätzliche Informationen

---

Review Checklist (für Reviewer)

• Code ist verständlich und wartbar
• Tests sind vorhanden und aussagekräftig
• Dokumentation ist ausreichend
• Keine offensichtlichen Security Issues
• Performance ist akzeptabel
• Breaking Changes sind gerechtfertigt und dokumentiert