Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
## What changed and why

## Dry-run result

- [ ] Triggered:
`gh workflow run safe-settings-sync.yml --repo IntegratedDynamic/admin --ref $BRANCH -f nop=true`
- [ ] Output reviewed — no unexpected diffs
- [ ] Known safe-settings bugs not triggered:
- `bypass_pull_request_allowances` not added to any suborg file
- `contexts:` uses `[]`, not a placeholder string
- No subdirectory added to `.github/suborgs/`
3 changes: 2 additions & 1 deletion CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,8 @@ Only declare what **changes** at each level — everything else is inherited via

Controls the safe-settings **process** (not individual repos):

- `restrictedRepos.exclude` — repos safe-settings will never touch (currently: `admin`, `.github`)
- `restrictedRepos.exclude` — repos safe-settings will never touch (currently: `.github` only —
`admin` is managed like any other repo)
- `configvalidators` — validate a single setting value (e.g. block admin collaborator permission)
- `overridevalidators` — validate when a suborg/repo overrides an org setting (e.g. block lowering
`required_approving_review_count` below org baseline)
Expand Down
3 changes: 0 additions & 3 deletions deployment-settings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,7 @@
# Add any repo that manages its own settings independently.
restrictedRepos:
exclude:
- admin # the settings repo itself
- .github # org-level .github repo
# safe-settings repo deleted — app runs from github/safe-settings via GitHub Actions
# gitops and infrastructure are now managed — remove to re-exclude

# configvalidators: validate a setting value in isolation.
# The script receives `baseconfig` (the setting being applied) and must return true/false.
Expand Down
Loading