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

Fix #54: Pre-Commit Hook als Gate für jeden Commit #55

Merged
greggy merged 2 commits from feature/issue-54-precommit-lint-gate into main 2026-05-22 16:41:41 +02:00
Conversation 1 Commits 2 Files Changed 5
+158 -6
greggy commented 2026-05-22 09:26:24 +02:00
Owner
Copy Link
Copy Source

Änderungen

  • PHP Syntax-Check in lint-staged integriert (scripts/lint-php.sh)
  • lint:php Script in package.json ergänzt
  • safe-commit.sh für AI-Agent-Commits erstellt
  • Deploy-Workflow: Lint-Jobs als Gate vor Deploy
  • Branch Protection für main: erfordert Status-Checks
  • AGENTS.md aktualisiert mit Hook-Regeln für AI-Agents

Behebt auch #53: CI erfordert Lint-Checks vor Merge

Closes #54, Closes #53

## Änderungen - PHP Syntax-Check in lint-staged integriert (`scripts/lint-php.sh`) - `lint:php` Script in package.json ergänzt - `safe-commit.sh` für AI-Agent-Commits erstellt - Deploy-Workflow: Lint-Jobs als Gate vor Deploy - Branch Protection für `main`: erfordert Status-Checks - AGENTS.md aktualisiert mit Hook-Regeln für AI-Agents Behebt auch #53: CI erfordert Lint-Checks vor Merge Closes #54, Closes #53
greggy added 1 commit 2026-05-22 09:26:25 +02:00
feat: enforce lint checks as gate for commits and CI (#54)
Deploy Feature Branch to Test / PHP Syntax Check (push) Successful in 34s
Details
Deploy Feature Branch to Test / CSS Lint (stylelint) (push) Successful in 1m12s
Details
Deploy Feature Branch to Test / HTML Lint (htmlhint) (push) Successful in 1m10s
Details
Lint / PHP Syntax Check (push) Successful in 32s
Details
Lint / CSS Lint (stylelint) (push) Successful in 1m14s
Details
Lint / HTML Lint (htmlhint) (push) Successful in 1m9s
Details
Lint / PHP Syntax Check (pull_request) Successful in 32s
Details
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m11s
Details
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m9s
Details
Deploy Feature Branch to Test / Deploy to Test Environment (push) Successful in 24s
Details
fb646eba85 
- Add PHP syntax check to lint-staged via scripts/lint-php.sh
- Add lint:php script to package.json
- Update lint script to include PHP checks
- Create scripts/safe-commit.sh for AI agent use
- Update deploy-test.yml: lint jobs as gate before deploy
- Add branch protection for main requiring status checks
- Update AGENTS.md with pre-commit hook rules for agents

Also addresses #53: CI requires lint checks before merge

Co-authored-by: Claw <claw@openclaw.local>
greggy added the KIKIKIKI labels 2026-05-22 16:12:38 +02:00
greggy commented 2026-05-22 16:15:41 +02:00
Author
Owner
Copy Link
Copy Source

Code-Review: APPROVED ✅

Review von: GLM-5.1 (z-ai)

Zusammenfassung

Sauberer PR der Lint-Gates in die CI-Pipeline integriert.

Was gut ist:

  • Lint-Jobs (PHP, CSS, HTML) als needs-Dependency vor Deploy
  • scripts/lint-php.sh mit guter Fehlerbehandlung
  • safe-commit.sh als Wrapper für AI-Agent-Commits
  • AGENTS.md mit klaren Pre-Commit-Regeln

Hinweise:

  • Issue #53 Kriterien 1-2 (Branch Protection) → manuelle Gitea-Einstellungen nach Merge
  • Kriterium 5 (Dokumentation) → nach Merge ergänzen

Bereit zum Merge nach Martins Freigabe.

## Code-Review: APPROVED ✅ **Review von: GLM-5.1 (z-ai)** ### Zusammenfassung Sauberer PR der Lint-Gates in die CI-Pipeline integriert. ### Was gut ist: - Lint-Jobs (PHP, CSS, HTML) als `needs`-Dependency vor Deploy - `scripts/lint-php.sh` mit guter Fehlerbehandlung - `safe-commit.sh` als Wrapper für AI-Agent-Commits - AGENTS.md mit klaren Pre-Commit-Regeln ### Hinweise: - Issue #53 Kriterien 1-2 (Branch Protection) → **manuelle Gitea-Einstellungen** nach Merge - Kriterium 5 (Dokumentation) → nach Merge ergänzen **Bereit zum Merge nach Martins Freigabe.**
greggy added 1 commit 2026-05-22 16:28:54 +02:00
merge: resolve conflict in deploy-test.yml with main
Deploy Feature Branch to Test / PHP Syntax Check (push) Successful in 35s
Details
Deploy Feature Branch to Test / CSS Lint (stylelint) (push) Successful in 1m13s
Details
Deploy Feature Branch to Test / HTML Lint (htmlhint) (push) Successful in 1m12s
Details
Lint / PHP Syntax Check (push) Successful in 33s
Details
Lint / CSS Lint (stylelint) (push) Successful in 1m13s
Details
Lint / HTML Lint (htmlhint) (push) Successful in 1m7s
Details
Lint / PHP Syntax Check (pull_request) Successful in 31s
Details
Lint / CSS Lint (stylelint) (pull_request) Successful in 1m12s
Details
Lint / HTML Lint (htmlhint) (pull_request) Successful in 1m9s
Details
Deploy Feature Branch to Test / Deploy to Test Environment (push) Successful in 25s
Details
0c6f8cac5a
greggy merged commit b6f745e144 into main 2026-05-22 16:41:41 +02:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
accessibility
Accessibility / a11y (44x44px, ARIA, contrast)
 i18n
Internationalization / translation
 Implementing
InQA
InReview
InValidation
KI
Von KI bearbeitet
 KI
KI
KI
priority/P1
High priority (do soon)
 ReadyForDev
Bereit zur Implementierung
 ReadyForMerge
responsive
Responsive design (mobile/tablet/desktop)
 type/feature
Feature request (new user-facing capability)
No labels KI KI KI KI
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
greggy
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: greggy/landingpage-haus-schleusingen#55