Skip to content

ci: trigger rusty-v8 releases from tags#21259

Merged
cconger merged 3 commits intomainfrom
cconger/rusty-v8-release-tags
May 5, 2026
Merged

ci: trigger rusty-v8 releases from tags#21259
cconger merged 3 commits intomainfrom
cconger/rusty-v8-release-tags

Conversation

@cconger
Copy link
Copy Markdown
Contributor

@cconger cconger commented May 5, 2026

Swap to tag based releasing and allow tags of type rusty-v8-v*.*.*

cconger added 2 commits May 5, 2026 22:37
Swap to tag based releasing and allow tags of type rusty-v8-v*.*.*
@cconger cconger requested a review from bolinfest May 5, 2026 22:46
Copy link
Copy Markdown
Contributor

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

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: cef1ce37b9

ℹ️ 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".

echo "Publishing is only allowed from ${DEFAULT_BRANCH}; current ref is ${GITHUB_REF_NAME}." >&2
exit 1

- name: Ensure release tag is new
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Badge Gate tag releases to the default branch

The new tag trigger publishes any matching tag without checking that the tag's commit is on the default branch. A tag pushed from an old or side-branch commit with the same v8 version can create the immutable rusty-v8-v<version> release first, producing artifacts that don't match main's checked-in checksum manifest and blocking the correct release.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is the feature we want :)

@cconger cconger merged commit 03d3403 into main May 5, 2026
29 checks passed
@cconger cconger deleted the cconger/rusty-v8-release-tags branch May 5, 2026 23:56
@github-actions github-actions Bot locked and limited conversation to collaborators May 5, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants