Skip to content

fix: restore missing release binaries#1254

Merged
jbedard merged 1 commit intobazel-contrib:mainfrom
acozzette:fix-release
Apr 29, 2026
Merged

fix: restore missing release binaries#1254
jbedard merged 1 commit intobazel-contrib:mainfrom
acozzette:fix-release

Conversation

@acozzette
Copy link
Copy Markdown
Contributor

The release process seems to have broken due to this bug in download-artifact: actions/download-artifact#455 We use the release_ruleset.yaml reusable workflow, which downloads all artifacts here. But since there is only one artifact (go-binaries.zip), we hit that bug which causes the binaries to appear directly in the working directory instead of in ./go-binaries.

This change fixes the problem by nesting everything inside a go-binaries directory inside go-binaries.zip. This way when the file is unzipped, all the binaries appear to be in the same place they were before.

Comment thread .github/workflows/release.yaml
The release process seems to have broken due to this bug in download-artifact:
actions/download-artifact#455 We use the
release_ruleset.yaml reusable workflow, which downloads all artifacts
[here](https://github.com/bazel-contrib/.github/blob/af28703814cf26522cf5e44d219efece91293123/.github/workflows/release_ruleset.yaml#L103).
But since there is only one artifact (`go-binaries.zip`), we hit that bug which
causes the binaries to appear directly in the working directory instead of in
`./go-binaries`.

This change fixes the problem by nesting everything inside a `go-binaries`
directory inside `go-binaries.zip`. This way when the file is unzipped, all the
binaries appear to be in the same place they were before.
@jbedard
Copy link
Copy Markdown
Collaborator

jbedard commented Apr 29, 2026

FYI @alexeagle @fmeum this was introduced in #1249 and caused the 3.3.0 release to not have the artifacts in it

@acozzette acozzette changed the title fix: fix the release process fix: restore missing release binaries Apr 29, 2026
@jbedard jbedard merged commit 84bd7e4 into bazel-contrib:main Apr 29, 2026
20 checks passed
@acozzette acozzette deleted the fix-release branch April 29, 2026 21:23
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.

4 participants