bazelbuild/bazel

docs: https://bazel.build/concepts/dependencies still refers to old style directory labels

Open

#15,574 创建于 2022年5月25日

在 GitHub 查看
 (0 评论) (0 反应) (0 负责人)Java (4,465 fork)batch import
P3help wantedteam-Documentationteam-Rules-APItype: feature request

仓库指标

Star
 (25,384 star)
PR 合并指标
 (平均合并 22天 20小时) (30 天内合并 77 个 PR)

描述

Description of the feature request:

https://bazel.build/concepts/dependencies#using-labels-reference-directories (and its pre-move equivalent https://docs.bazel.build/versions/main/build-ref.html#label_directory) still refer to labels pointing to directories with the old syntax like data = ["testdata/."], mentioning them being present in the bazel source code. As far as I can tell there are no such references in the bazel codebase anymore. In addition to that since https://github.com/bazelbuild/bazel/issues/1025 has been solved directories should now be able to be referred to with the regular label syntax as if they were files.

The documentation as is makes it confusing as it implies labels referring to directories need to use that outdated syntax and have caveats which I don't think are true anymore.

What underlying problem are you trying to solve with this feature?

Incorrect documentation.

Which operating system are you running Bazel on?

macOS

What is the output of bazel info release?

release 5.0.0- (@non-git)

If bazel info release returns development version or (@non-git), tell us how you built Bazel.

From nix: https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/tools/build-managers/bazel/bazel_5/default.nix

What's the output of git remote get-url origin; git rev-parse master; git rev-parse HEAD ?

git@github.com:bazelbuild/bazel.git
c282526c071389cd6f88cb77565283b257316267
0f5dc111be06b2ee8694640f400b58e12bfa5fea

Have you found anything relevant by searching the web?

No.

Any other information, logs, or outputs that you want to share?

No response

贡献者指南