Repository navigation
governance: Address CodeRabbit feedback and remove locked-files-guard - #435
Conversation
- Clarify BEM-only CSS architecture in README overview - Document that Tailwind tooling is retained for Make but not used in components - Update Phase 6 checkpoint to accurately reflect Tailwind status - Note outstanding T046 Styling section requirement - Document T029 protected files guard approval process integration Addresses CodeRabbit review comments on: • README.md line 25: CSS architecture clarity • tasks.md line 378: Phase 6 Tailwind status • tasks.md line 268-270: T029 approval documentation Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Remove the Constitution Principle II enforcement workflow that blocks PRs modifying protected files without approved change requests. This workflow is redundant with the protected-files-guard.yml workflow and the manual change-request approval process via issue #428. Eliminates unnecessary CI complexity while maintaining governance through: - Protected file restrictions via GitHub branch protection - Manual change-request approval via [TEMPLATE-UPDATE-REQUEST] issues - CODEOWNERS review requirements Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
✅ Deploy Preview for thisonetimeonacid ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
PR Template Routing 📋Branch prefix: build Suggested template: pr_default.md Type: General PR Canonical templates are managed in lightspeedwp/.github:
|
Drift Detection ✅✅ No drift detected. Configuration is canonical. Canonical governance files are correctly centralized in lightspeedwp/.github. |
|
✅ Workflow Syntax Validation All workflows in Result: All workflows passed validation |
|
👤 Human-authored PR — awaiting review and approval. Checklist before merge:
|
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing: 3
✨ Finishing Touches
Comment |
Summary
Address remaining CodeRabbit review feedback on Spec-001 governance documentation and simplify CI/CD governance by removing redundant locked-files-guard workflow.
Changes
CodeRabbit Review Fixes
Workflow Refactoring
Related Issues
Relates to #346 (Spec-001 Phase 2-5), #428 [TEMPLATE-UPDATE-REQUEST]
🤖 Generated with Claude Code