Skip to content

ci(release): restrict core release trigger to v[0-9]* tags#249

Merged
avanelsas merged 1 commit into
mainfrom
fix/release-trigger-glob
May 26, 2026
Merged

ci(release): restrict core release trigger to v[0-9]* tags#249
avanelsas merged 1 commit into
mainfrom
fix/release-trigger-glob

Conversation

@avanelsas
Copy link
Copy Markdown
Owner

Summary

One-line fix to the trigger glob in `.github/workflows/release.yml`.

The current `v*` glob accidentally matches adapter tags that happen to start with 'v' — `vue-v0.1.0` being the obvious one. When that tag was pushed during the Svelte/Solid/Vue adapter release sequence (PR #248 follow-up), the main `release` workflow queued alongside `release-vue` and would have overwritten the Vue GitHub Release name had it not been cancelled in flight (`gh run cancel 26450464475`).

Tightening to `v[0-9]` restricts the core release to genuine semver tags (`v3.3.0`, `v3.4.0-rc.1`, etc.) and isolates each adapter release to its dedicated workflow. No other adapter tag patterns are affected (`react-v`, `angular-v*`, `svelte-v*`, `solid-v*` start with letters, not 'v').

Test plan

  • Diff is a 1-line glob change plus an explanatory comment
  • No other workflow file uses the `v*` pattern (verified — adapter workflows use `-v*` which already discriminates)
  • Live verification (follow-up): the next core release (`v3.3.0`) should trigger `release.yml` exactly once; a future adapter tag should not.

🤖 Generated with Claude Code

The `v*` glob accidentally matched adapter tags that happen to start
with 'v', most notably `vue-v0.1.0`. When that tag was pushed during
the adapter release sequence, the main `release` workflow queued
alongside `release-vue` and would have overwritten the Vue GitHub
Release name had it not been cancelled.

Tightening the glob to `v[0-9]*` restricts the core release to
genuine semver tags (v3.3.0, v3.4.0-rc.1, etc.) and isolates each
adapter release to its dedicated workflow.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@avanelsas avanelsas merged commit 5bbb71b into main May 26, 2026
1 check passed
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