Skip to content

ci: remove the automatic release back-merge (backport #5179) - #5180

Merged
kixelated merged 1 commit into
releasefrom
backport/remove-back-merge
Oct 10, 2026
Merged

kixelated merged 1 commit into
releasefrom
backport/remove-back-merge

Conversation

@kixelated

Copy link
Copy Markdown
Collaborator

Problem

Backport of #5179. release's copy of the Back-merge workflow runs on every push to release and opens a PR merging release into main. Once #5179 deletes Check's land job on main, nothing lands those PRs. The maintainer decided (2026-10-10):

remove it, we can just cherry pick from main to release

once merge release into main before a semver release

Approach

Delete .github/workflows/back-merge.yml, sh/gh/back-merge.sh, the just gh back-merge recipe, and the Back-merge entry in alert.yml. release's check.yml has no land job, and its CONTRIBUTING.md never described the back-merge, so nothing else changes here. Not a literal cherry-pick, since #5179 also touches files that differ on release.

Land it with a merge commit, per CONTRIBUTING.md.

Ordering

The main PR (#5179) must merge only after #5176 has merged, or after the maintainer merges #5176 by hand. This backport can land in either order; its own push to release no longer carries the workflow, so it opens no back-merge.

Impact

  • No public API or wire change. CI only.

(Written by Claude Opus 5.5)

🤖 Generated with Claude Code

release's copy of the Back-merge workflow runs on every push to release,
so it must go here too. Drop the workflow, its script and recipe, and the
Alert entry.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kixelated
kixelated marked this pull request as ready for review October 10, 2026 12:27
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-10T12:30:00.806242Z 497a9dc Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 497a9dc807

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -1,51 +0,0 @@
name: Back-merge

# Every push to `release` (a cut, a backport, or release-plz's version bump)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Update the release plan with the new merge policy

With this workflow removed, quest/m0/branch-flip.md:49-51, :64-68, and :84-86 still state that every publish is automatically back-merged and require verifying that behavior. Anyone consulting the durable release plan could therefore wait for automation that no longer exists instead of performing the newly required manual merge before a semver release; update those decisions and verification steps in this change.

AGENTS.md reference: AGENTS.md:L28-L28

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Not changing this here. Quests are planned on main, which already deleted quest/m0/branch-flip.md as finished in #4899; release only carries a stale copy from the last cut. Editing it on release would turn into a modify/delete conflict at the next pre-release merge of release into main, and the next cut drops the file anyway. The current policy lives in main's CONTRIBUTING.md via #5179.

(Written by Claude Opus 5.5)

@kixelated

Copy link
Copy Markdown
Collaborator Author

Merge summary:

(Written by Claude Opus 5.5)

@kixelated
kixelated merged commit 88fbe72 into release Oct 10, 2026
5 checks passed
@kixelated
kixelated deleted the backport/remove-back-merge branch October 10, 2026 12:35
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