Alex Eagle
Alex Eagle
I think @kormide will be a good code reviewer for that.
I think the style guide at https://docs.bazel.build/versions/main/skylark/deploying.html mostly dictates this. But I could take this issue to suggest that we do more to maintain the style guide for rules authors....
We could take a page like https://docs.bazel.build/versions/main/skylark/deploying.html and update from there.
It looks like there's movement from the Bazel team at Google on this: https://github.com/bazelbuild/proposals/pull/247
Note, the upstream docs are being extended: https://github.com/bazelbuild/bazel/pull/15800
@limdor from some discussion in the SIG meeting - is this going to mean that rule implementations and macros will now fail if they use `native.glob` and it returns no...
@wyverald says that it's theoretically possible to make two different `glob` symbols, though it's not that way today.
Yeah our discussion was just that it's going to be a long, painful slog to identify all the locations where the `allow_empty=True` needs to be added across the whole ecosystem,...
Bazel 7 will have bzlmod enabled by default, so if the SIG wants to fund this work, I think it's the right time now. Also @rickeylev indicated in an email...
For Go see https://github.com/bazelbuild/rules_go/issues/3717