Skip to content

docs: require approval before release merges - #290

Merged
jlitola merged 1 commit into
mainfrom
jlitola/release-merge-human-approval
Aug 18, 2026
Merged

docs: require approval before release merges#290
jlitola merged 1 commit into
mainfrom
jlitola/release-merge-human-approval

Conversation

@jlitola

@jlitola jlitola commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • make release preparation stop after opening the release PR
  • require a separate, explicit human approval after that PR exists before merging
  • forbid treating an earlier release request or auto-merge as approval
  • keep canonical release guidance aligned with a regression test

Validation

  • bun run plugins:generate
  • bun run plugins:check
  • bun run lint
  • bun run format
  • bun test (2,904 tests)
  • bun run build

Release impact

None. This changes release governance and carries an all-none fragment for later consumption.

Make release preparation stop at an open PR and require a separate post-PR human merge approval across repository guidance. Add a regression test for instruction parity.
@jlitola jlitola added the documentation Improvements or additions to documentation label Aug 18, 2026
@jlitola
jlitola merged commit 3540f1f into main Aug 18, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant