greggy/landingpage-haus-schleusingen
1
0
Fork 0
Code Issues 2 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

Ä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)
All checks were successful
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
KI
KI
KI
KI
 labels 2026-05-22 16:12:38 +02:00
greggy commented 2026-05-22 16:15:41 +02:00
Author
Owner
Copy Link

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
All checks were successful
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
Implementing
InQA
InReview
InValidation
KI
Von KI bearbeitet
KI
KI
KI
ReadyForDev
Bereit zur Implementierung
ReadyForMerge
No Label
KI
KI
KI
KI
Milestone
No items
No Milestone
Projects
Clear projects
No project
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
No description provided.
Delete Branch "feature/issue-54-precommit-lint-gate"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?