Skip to content

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

Merged
mergify[bot] merged 1 commit into
mainfrom
devs/sileht/mrgfy-11553-setup-cli-major-tag-race/move-major-tag-release-own-run-cut--cf242be7
Nov 5, 2026
Merged

mergify[bot] merged 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 deployed to Mergify Merge Protections November 4, 2026 01:30 Active
@mergify

mergify Bot commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 6 merge protections satisfied — ready to merge.

Show 6 satisfied protections

🟢 🤖 Continuous Integration

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

🟢 👀 Review Requirements

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

🟢 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)(?:\(.+\))?!?:

🟢 🔎 Reviews

  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

🟢 📕 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
@sileht
sileht marked this pull request as ready for review November 4, 2026 19:34
@mergify
mergify Bot merged commit 284904f into main Nov 5, 2026
9 checks passed
@mergify

mergify Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

✅ Merged into main as 284904f at 07:55 UTC

Less than a minute in the queue, with no time running CI.

So far

  • ✅ Entered the queue Thu 5 Nov, 07:55 · rule default · triggered by merge protections
  • ✅ Checks skipped · PR is already up-to-date
  • ✅ Merged 07:55 · 284904f · squash
7 merge conditions met
  • title ~= ^(fix|feat|internal|docs|style|refactor|perf|test|build|ci|chore|revert|ui)(?:\(.+\))?!?: · required by Merge Protections rule Enforce conventional commit
  • any of · required by Merge Protections rule 👀 Review Requirements:
    • #approved-reviews-by>=1
    • author = dependabot[bot]
    • author = mergify-ci-bot
    • author = renovate[bot]
  • body ~= (?ms:.{48,}) · required by Merge Protections rule 📕 PR description
  • #changes-requested-reviews-by = 0 · required by Merge Protections rule 🔎 Reviews
  • #review-requested = 0 · required by Merge Protections rule 🔎 Reviews
  • #review-threads-unresolved = 0 · required by Merge Protections rule 🔎 Reviews
  • check-success=all-greens · required by Merge Protections rule 🤖 Continuous Integration

Times in UTC · dashboard · how the merge queue works

@mergify
mergify Bot deleted the devs/sileht/mrgfy-11553-setup-cli-major-tag-race/move-major-tag-release-own-run-cut--cf242be7 branch November 5, 2026 07:55
@mergify mergify Bot added queued and removed queued labels Nov 5, 2026

This branch was successfully deployed

1 active 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.

2 participants