Skip to content

build(release): build all release deliverables in presubmit - #4281

Merged
rickeylev merged 2 commits into
bazel-contrib:mainfrom
rickeylev:test_release_workflows_presubmit
Oct 19, 2026
Merged

rickeylev merged 2 commits into
bazel-contrib:mainfrom
rickeylev:test_release_workflows_presubmit

Conversation

@rickeylev

Copy link
Copy Markdown
Collaborator

Release workflow bugs previously went unnoticed until cutting a release
because artifact creation was coupled with publishing.

Move runfiles wheel building, archive/notes generation, and artifact
packaging into release_build_artifacts.yaml with contents: read
permissions, trigger it on pull_request, and have
release_publish.yaml download and publish the built artifacts.

Release workflow bugs previously went unnoticed until cutting a release
because artifact creation was coupled with publishing.

Move runfiles wheel building, archive/notes generation, and artifact
packaging into `release_build_artifacts.yaml` with `contents: read`
permissions, trigger it on `pull_request`, and have
`release_publish.yaml` download and publish the built artifacts.
…ert RELEASING.md

Git < 2.35 (on Debian 11 and Ubuntu 22.04) does not support the :tags=true option in %(describe:tags=true). Always assert _COMMIT_PRIVATE expansion across all Git versions and only check _VERSION_PRIVATE when supported by the host git. Also revert unneeded presubmit notes from RELEASING.md.
@rickeylev
rickeylev marked this pull request as ready for review October 19, 2026 03:27
@rickeylev
rickeylev requested a review from aignas as a code owner October 19, 2026 03:27
@rickeylev
rickeylev merged commit 9f3ab06 into bazel-contrib:main Oct 19, 2026
18 checks 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