Skip to content

feat: add branch protection rules, PR check workflow, and CODEOWNERS - #30

Merged
CodeWithJuber merged 1 commit into
mainfrom
claude/gap-analysis-mizan-qalb-65EfV
Feb 28, 2026
Merged

feat: add branch protection rules, PR check workflow, and CODEOWNERS#30
CodeWithJuber merged 1 commit into
mainfrom
claude/gap-analysis-mizan-qalb-65EfV

Conversation

@CodeWithJuber

Copy link
Copy Markdown
Owner
  • Add pr-check.yml workflow: lint, test, frontend build checks on PRs to main/master
  • Add setup-branch-protection.sh script to configure GitHub branch rules via gh CLI
  • Add CODEOWNERS file assigning @CodeWithJuber as default reviewer
  • Add make protect target to Makefile

Branch protection rules (applied via script):

  • Require PR with 1 approval before merging
  • Dismiss stale reviews on new pushes
  • Require status checks: Lint, Test, Frontend Build
  • Require branch up to date before merge
  • Require conversation resolution
  • Block force pushes and branch deletion
  • Enforce for admins

https://claude.ai/code/session_01JGamoWGB9PN39TCpi3F6J1

Summary

Changes

Test Plan

  • Tests pass (make test)
  • Linting passes (make lint)
  • Manually tested

Related Issues

- Add pr-check.yml workflow: lint, test, frontend build checks on PRs to main/master
- Add setup-branch-protection.sh script to configure GitHub branch rules via gh CLI
- Add CODEOWNERS file assigning @CodeWithJuber as default reviewer
- Add `make protect` target to Makefile

Branch protection rules (applied via script):
- Require PR with 1 approval before merging
- Dismiss stale reviews on new pushes
- Require status checks: Lint, Test, Frontend Build
- Require branch up to date before merge
- Require conversation resolution
- Block force pushes and branch deletion
- Enforce for admins

https://claude.ai/code/session_01JGamoWGB9PN39TCpi3F6J1
@CodeWithJuber
CodeWithJuber merged commit 5e010b1 into main Feb 28, 2026
1 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants