greggy/landingpage-haus-schleusingen
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Hooks: PHPUnit in Pre-Commit (#67) #70

Merged
greggy merged 3 commits from feature/issue-67-phpunit-precommit into main 2026-06-04 02:41:15 +02:00
Conversation 0 Commits 3 Files Changed 3
+85 -13
3 Commits
Author SHA1 Message Date
Hermes 3f1f0f5788 fix(hooks): use array-based file iteration for safety check
Deploy Feature Branch to Test / PHP Syntax Check (push) Successful in 58s
Details
Lint / PHP Syntax Check (push) Successful in 1m0s
Details
Deploy Feature Branch to Test / HTML Lint (htmlhint) (push) Successful in 1m39s
Details
Deploy Feature Branch to Test / CSS Lint (stylelint) (push) Successful in 1m42s
Details
Deploy Feature Branch to Test / Deploy to Test Environment (push) Successful in 23s
Details
Lint / HTML Lint (htmlhint) (push) Successful in 1m16s
Details
Lint / CSS Lint (stylelint) (push) Successful in 1m21s
Details
Lint / PHP Syntax Check (pull_request) Successful in 31s
Details
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m13s
Details
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m4s
Details 
mapfile + array prevents word-splitting issues with filenames
containing spaces or other shell-special characters. Affects both
the affected-files listing and the stale-index safety check.

Without this fix, a filename like 'My Module.php' would be split
into 'My' and 'Module.php', causing the disk-existence check to
look for wrong paths.
 2026-06-04 00:15:21 +00:00
Hermes b0f769d186 feat(hooks): run PHPUnit in pre-commit hook (#67)
Deploy Feature Branch to Test / PHP Syntax Check (push) Successful in 34s
Details
Lint / PHP Syntax Check (push) Successful in 57s
Details
Deploy Feature Branch to Test / CSS Lint (stylelint) (push) Successful in 1m41s
Details
Deploy Feature Branch to Test / HTML Lint (htmlhint) (push) Successful in 1m39s
Details
Lint / CSS Lint (stylelint) (push) Successful in 1m39s
Details
Lint / HTML Lint (htmlhint) (push) Successful in 1m8s
Details
Deploy Feature Branch to Test / Deploy to Test Environment (push) Successful in 30s
Details 
Erweitert den Husky pre-commit-Hook um einen PHPUnit-Schritt.
Ausserdem wird scripts/safe-commit.sh aktualisiert, damit das
Safety-Net dieselbe Logik wie der Hook verwendet (kein doppelter Code).

Vorher: Hook rief nur 'npx lint-staged' auf.
Nachher: Hook ruft scripts/pre-commit-checks.sh auf, das
  - lint-staged ausfuehrt (unveraendert)
  - PHPUnit nur dann ausfuehrt, wenn PHP-relevante Dateien
    gestaged sind (Performance-Optimierung: Issue #67 Anforderung)
  - Bei Test-Fehler den Commit mit Exit-Code 1 abbricht

Closes #67
 2026-06-03 23:42:31 +00:00
Hermes 85cf4f3b03 chore(hooks): add shared pre-commit-checks script 
Extrahiert die Pre-Commit-Checks (lint-staged + PHPUnit) in ein
gemeinsames Script, das sowohl vom Husky-Hook (.husky/pre-commit)
als auch von scripts/safe-commit.sh aufgerufen wird.

Logik:
- lint-staged laeuft immer (HTML/CSS/JS/JSON/MD/PHP-Syntax)
- PHPUnit laeuft nur, wenn PHP-relevante Dateien gestaged sind
  (*.php, phpunit.xml, composer.json, composer.lock)
- Safety-Check: alle gestaged PHP-Dateien muessen auf Disk existieren
- Bei Test-Fehler wird der Commit mit Exit-Code != 0 abgebrochen
- Composer-Deps werden nur bei Bedarf installiert (Cache-Hit)
 2026-06-03 23:42:22 +00:00