Skip to content

Fast-forward master to origin/dev instead of a release PR - #171

Merged
AdamSpitz merged 2 commits into
devfrom
chore/ff-promote-master
Sep 8, 2026
Merged

Fast-forward master to origin/dev instead of a release PR#171
AdamSpitz merged 2 commits into
devfrom
chore/ff-promote-master

Conversation

@AdamSpitz

Copy link
Copy Markdown
Owner

Turn off the GitHub PR requirement on master so release can be a real fast-forward of origin/dev onto master (same SHA, no merge commit, no back-merge).

scripts/protect-branches.sh is already applied on GitHub: master still forbids force-push and delete; dev is unchanged.

Use scripts/promote-dev-to-master.sh. One leftover from PR #170: trees match but master is not an ancestor of dev, so the first catch-up needs a one-time force (--reset-master-to-dev after temporarily allowing force-push).

GitHub no longer requires a PR on master; force-push stays off so the
branch can only move forward. Local hooks allow pushing master only when
the new SHA is current origin/dev.
Also fix review-gate docs: review-received is required on dev only.

@AdamSpitz AdamSpitz left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review receipt.

Reviewed the promote-path change: master is a FF of origin/dev, no release PR. Hooks, protect-branches.sh, AGENTS.md, branching.md, and the PR template agree. No findings.

Reviewed-with: grok
Reviewed-files: 14
Reviewed-commit: 6f7a46a

@AdamSpitz
AdamSpitz merged commit 8ad5bb4 into dev Sep 8, 2026
7 checks passed
@AdamSpitz
AdamSpitz deleted the chore/ff-promote-master branch September 8, 2026 15:20
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.

1 participant