Skip to content

Commit d822422

Browse files
committed
Chore: Remove protect-main.sh; main protection is the maintainer's settings (#126)
1 parent 0ff7a07 commit d822422

2 files changed

Lines changed: 5 additions & 44 deletions

File tree

‎tools/release/README.md‎

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,9 @@ ignored and overwritten.
1616
`GITHUB_TOKEN`. This needs the repository setting *Actions -> General -> Allow GitHub
1717
Actions to create and approve pull requests*. No PAT is needed; an optional
1818
`RELEASE_PR_TOKEN` secret is used for the PR step when present.
19-
3. Merging that PR is the only way to change `main`. Branch protection (below) makes `main`
20-
PR-only and lets only administrators merge; the maintainer merges the `release` PR and
21-
checks that its head is `release` (maintainer decision 2026-10-03: no source-branch
22-
workflow, settings only).
19+
3. Merging that PR is the only way to change `main`. The maintainer protects `main` in the
20+
repository settings (locked) and merges the `release` PR after checking that its head is
21+
`release` (maintainer decision 2026-10-03: settings only, no scripts or workflows for it).
2322
4. On push to `main`, `pages.yml` deploys `docs/guide/` to GitHub Pages.
2423

2524
## What is dropped on release
@@ -42,8 +41,5 @@ It refuses to run while `release` is checked out.
4241

4342
## Protecting main
4443

45-
tools/release/protect-main.sh # print what would be applied
46-
tools/release/protect-main.sh --apply # apply via gh api (needs admin)
47-
48-
This requires a pull request (no approving review needed), blocks direct push, force-push and
49-
deletion, leaves the branch unlocked, and restricts merging to administrators.
44+
`main`'s protection is set by the maintainer in the repository settings (the branch is locked;
45+
the maintainer merges the `release` pull request). Agents do not script or change it.

‎tools/release/protect-main.sh‎

Lines changed: 0 additions & 35 deletions
This file was deleted.

0 commit comments

Comments
 (0)