-
Notifications
You must be signed in to change notification settings - Fork 4
bazelisk support #19
Comments
Hi! Isn't bazelisk made to work with multiple workspaces that use different bazel versions locally? For a given workspace you always know the version of bazel, so you can always use it in the actions file. Why would you use bazelisk in CI? |
bazelisk has some cool features such as |
I am not sure that's the same use case as building / testing code with a known bazel version. @curtismuntz what's your use-case for bazelisk in ci? |
Thank you for your input! I thought a bit about this, and I no longer see any reason for this action, see #25 |
A lot of repositories are using bazelisk(an official bazelbuild project) to trivially change between bazel versions. It is instrumented off of a
.bazelversion
file in the workspace root. It'd be useful to have an action that keys off of this existing tool.This would allow something like the following to support literally any previous/upcoming release of bazel, along with all of the amazing features provided by bazelisk:
I can make a PR that implements it very trivially, but figured I'd ask your thoughts on supporting this here. If this goes beyond the scope of this project, I can make a spin-off action called
bazelisk-action
as well.Thanks!
The text was updated successfully, but these errors were encountered: