Use Bazelisk instead of Bazel in GitHub Actions.#504
Use Bazelisk instead of Bazel in GitHub Actions.#504PiotrSikora wants to merge 2 commits intoproxy-wasm:mainfrom
Conversation
bazel-contrib/setup-bazel installs only "bazelisk" binary, not "bazel". The existing jobs worked on hosted GitHub runners, since both binaries are preinstalled, but "bazel" is not available when testing locally. Signed-off-by: Piotr Sikora <code@piotrsikora.dev>
|
This is blocked on #492 (s390x), since its Docker image doesn't have |
|
Any argument against closing this PR? It will be forever blocked since we have decided to continue using s390x in CI. |
|
Ignoring s390x for a moment, this PR attempted to address 2 issues: (1) We're using None of those are real problems on GitHub Actions, because the default container images have both tools preinstalled, and However, this breaks when testing locally using the |
bazel-contrib/setup-bazel installs only "bazelisk" binary, not "bazel".
The existing jobs worked on hosted GitHub runners, since both binaries
are preinstalled, but "bazel" is not available when testing locally.