Skip to content

Rules sol#58

Draft
alexeagle wants to merge 3 commits intodivergencetech:mainfrom
alexeagle:rules_sol
Draft

Rules sol#58
alexeagle wants to merge 3 commits intodivergencetech:mainfrom
alexeagle:rules_sol

Conversation

@alexeagle
Copy link
Contributor

No description provided.

This ensures that all developers have the same version of bazel as the CI machine.
Especially important if we decide to upgrade the version used, so devs aren't broken by missing features.
Note, Bazelisk, as well as Bazel's wrapper script you get from the installer, will check this file to prevent a wrong Bazel from running.
If we want a stronger guarantee, we can follow
https://github.com/bazelbuild/bazelisk#ensuring-that-your-developers-use-bazelisk-rather-than-bazel
Avoids version skew where some rule transitively installs an older bazel-skylib
Copy link

@gregmagolan gregmagolan left a comment

Choose a reason for hiding this comment

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

🌮

@gregmagolan
Copy link

Whoops. Nevermind the stamp. I thought I was stamping aspect-build/rules_sol#1.

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.

2 participants