Skip to content

ci: move the major tag to the release its own run just cut - #26

Draft
sileht wants to merge 1 commit into
mainfrom
devs/sileht/mrgfy-11553-setup-cli-major-tag-race/move-major-tag-release-own-run-cut--cf242be7
Draft

sileht wants to merge 1 commit into
mainfrom
devs/sileht/mrgfy-11553-setup-cli-major-tag-race/move-major-tag-release-own-run-cut--cf242be7

Conversation

@sileht

@sileht sileht commented Nov 4, 2026

Copy link
Copy Markdown
Member

The push run of #25 released
v1.4.0 and then left v1 on v1.3.0: the major-tag step listed the
releases right after creating v1.4.0, the list did not show it yet, and
the step logged "v1 already points at v1.3.0". Every
Mergifyio/setup-cli@v1 user stayed on the old default mergify-cli.

This is not a rare race. On a throwaway repository, the releases list
missed a release created just before in 24 of 25 tries, and took 0.6 to
1.8 s to show it, while refs/tags/ resolved to the right commit
25 times out of 25.

The step now counts the release its run cut, or the published release
that triggered it, on its own before picking the newest, whatever the
list returns. A patch on an older minor still never moves the tag
backwards, and a dispatched tag still only picks the major. The commit
is resolved from refs/tags/, so a branch with the same name cannot
be picked.

Merging this is a push to main, so its run points v1 at v1.4.0 if v1 is
still on v1.3.0 by then.

Fixes MRGFY-11553

Co-Authored-By: Claude Opus 5.5 noreply@anthropic.com

The push run of #25 released
v1.4.0 and then left v1 on v1.3.0: the major-tag step listed the
releases right after creating v1.4.0, the list did not show it yet, and
the step logged "v1 already points at v1.3.0". Every
`Mergifyio/setup-cli@v1` user stayed on the old default mergify-cli.

This is not a rare race. On a throwaway repository, the releases list
missed a release created just before in 24 of 25 tries, and took 0.6 to
1.8 s to show it, while refs/tags/<tag> resolved to the right commit
25 times out of 25.

The step now counts the release its run cut, or the published release
that triggered it, on its own before picking the newest, whatever the
list returns. A patch on an older minor still never moves the tag
backwards, and a dispatched tag still only picks the major. The commit
is resolved from refs/tags/<tag>, so a branch with the same name cannot
be picked.

Merging this is a push to main, so its run points v1 at v1.4.0 if v1 is
still on v1.3.0 by then.

Fixes MRGFY-11553

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Change-Id: Icf242be7761ae1385adc264082c2ce19cb4f2d16
@mergify
mergify Bot had a problem deploying to Mergify Merge Protections November 4, 2026 01:30 Failure
@mergify

mergify Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 2 of 6 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 👀 Review Requirements 👀 reviews
🔴 🔎 Reviews 👀 reviews
🟢 🤖 Continuous Integration —
🟢 Enforce conventional commit —
🟢 📕 PR description —
🟢 🚦 Auto-queue —

🔴 👀 Review Requirements

Waiting for

  • #approved-reviews-by>=1
This rule is failing.
  • any of:
    • #approved-reviews-by>=1
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]

🔴 🔎 Reviews

Waiting for

  • #review-requested = 0
This rule is failing.
  • #review-requested = 0
  • #changes-requested-reviews-by = 0
  • #review-threads-unresolved = 0

Show 4 satisfied protections

🟢 🤖 Continuous Integration

  • all of:
    • check-success=all-greens

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?:

🟢 📕 PR description

  • body ~= (?ms:.{48,})

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@mergify
mergify Bot requested a review from a team November 4, 2026 01:30

This branch had an error being deployed

1 failed deployment
Mergify Merge Protections — 3184260b Deployed Nov 4, 2026 by mergify[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant