bazelbuild / bazelisk

A user-friendly launcher for Bazel.
Apache License 2.0
1.95k stars 300 forks source link

Split lines of .bazelversion and return first nonempty. #576

Closed API92 closed 1 month ago

API92 commented 1 month ago

Split lines of .bazelversion and return first nonempty in bazelisk.py to support multiline .bazelversion in tensorflow.

fweikert commented 1 month ago

I think if we support multi-line files, then we should also skip comments. However, this can be done in a future PR, so I'm approving now.