-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bazelisk] Use Bazel 7.1.2 #349
[bazelisk] Use Bazel 7.1.2 #349
Conversation
68d91c7
to
d9b093b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+@jwnimmer-tri Could I request your review here?
\cc @calderpg-tri @IanTheEngineer
Reviewable status: 0 of 7 files reviewed, all discussions resolved (waiting on @jwnimmer-tri)
Seems like the kind of thing that's fine for a self-rubber-stamp. |
d9b093b
to
44c73a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good, thanks! +@EricCousineau-TRI
Reviewable status: 0 of 10 files reviewed, all discussions resolved (waiting on @EricCousineau-TRI and @jwnimmer-tri)
install_prereqs: Use Bazelisk environ.bzl: Use workaround for attr.string_list() from Anzu Co-Authored-By: Jeremy Nimmer <[email protected]>
44c73a1
to
af06800
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 7 of 7 files at r1, 3 of 3 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @jwnimmer-tri)
To match Anzu
This change is