Skip to content

Commit 92266f9

Browse files
authored
fix: complete required files list and add missing override (#14)
## Summary - Add 4 missing files to `required_files` in baseline.json that the global CLAUDE.md requires on every repo: `CLAUDE.md`, `.coderabbit.yaml`, `.github/copilot-instructions.md`, `.github/PULL_REQUEST_TEMPLATE.md` - Add `cloud-architecture-course` status check contexts to overrides.json (was the only managed repo without an override) - Update README and quality-checks to match ## Test plan - [ ] Verify quality-checks structure validation passes - [ ] Run dry-run sync to confirm new files are checked across repos - [ ] Confirm cloud-architecture-course override matches its actual checks <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Expanded repository structure validation requirements to include additional configuration files. * Updated documentation to reflect new required repository files. * Added branch protection configuration for an additional repository, including required status checks for code quality and validation. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
1 parent 64be5db commit 92266f9

4 files changed

Lines changed: 24 additions & 3 deletions

File tree

.github/workflows/quality-checks.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,9 @@ jobs:
6262
MISSING=0
6363
for file in LICENSE README.md .gitignore CODEOWNERS CONTRIBUTING.md \
6464
SECURITY.md CLAUDE.md .pre-commit-config.yaml \
65-
.github/dependabot.yml config/baseline.json \
66-
config/overrides.json; do
65+
.coderabbit.yaml .github/copilot-instructions.md \
66+
.github/dependabot.yml .github/PULL_REQUEST_TEMPLATE.md \
67+
config/baseline.json config/overrides.json; do
6768
if [ ! -f "$file" ]; then
6869
echo "MISSING: $file"
6970
MISSING=$((MISSING + 1))

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,8 +137,12 @@ report (not auto-created, since content is repo-specific).
137137
| `CODEOWNERS` | Assign default reviewers |
138138
| `CONTRIBUTING.md` | Contribution guidelines |
139139
| `SECURITY.md` | Vulnerability disclosure policy |
140+
| `CLAUDE.md` | Claude Code project instructions |
140141
| `.pre-commit-config.yaml` | Local linting and validation |
142+
| `.coderabbit.yaml` | CodeRabbit auto-review configuration |
143+
| `.github/copilot-instructions.md` | Copilot code review instructions |
141144
| `.github/dependabot.yml` | Automated dependency updates |
145+
| `.github/PULL_REQUEST_TEMPLATE.md` | PR checklist template |
142146

143147
### Metadata Checks (Advisory)
144148

config/baseline.json

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,11 @@
9797
"CODEOWNERS",
9898
"CONTRIBUTING.md",
9999
"SECURITY.md",
100+
"CLAUDE.md",
100101
".pre-commit-config.yaml",
101-
".github/dependabot.yml"
102+
".coderabbit.yaml",
103+
".github/copilot-instructions.md",
104+
".github/dependabot.yml",
105+
".github/PULL_REQUEST_TEMPLATE.md"
102106
]
103107
}

config/overrides.json

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,18 @@
2626
}
2727
}
2828
},
29+
"cloud-architecture-course": {
30+
"branch_protection": {
31+
"required_status_checks": {
32+
"contexts": [
33+
"Markdown Linting",
34+
"YAML Validation",
35+
"Validate Repository Structure",
36+
"Security Scan"
37+
]
38+
}
39+
}
40+
},
2941
"aws-organization-governance": {
3042
"branch_protection": {
3143
"required_status_checks": {

0 commit comments

Comments
 (0)