Skip to content

Commit a145ef0

Browse files
tetrominocomius
authored andcommitted
Make only root test_deps externally visible
All other test_deps targets implicitly require the root BUILD file for the `license` target for their default_applicable_licenses; therefore, users should only depend on the root test_deps target.
1 parent 27d429d commit a145ef0

File tree

3 files changed

+3
-1
lines changed

3 files changed

+3
-1
lines changed

lib/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ filegroup(
9999
name = "test_deps",
100100
testonly = True,
101101
srcs = ["BUILD"] + glob(["*.bzl"]),
102+
visibility = ["//:__subpackages__"], # Needs skylib's root BUILD file for default_applicable_licenses
102103
)
103104

104105
# The files needed for distribution

rules/BUILD

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ filegroup(
6969
srcs = [
7070
"BUILD",
7171
] + glob(["*.bzl"]),
72+
visibility = ["//:__subpackages__"], # Needs skylib's root BUILD file for default_applicable_licenses
7273
)
7374

7475
# The files needed for distribution

toolchains/unittest/BUILD

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ filegroup(
5656
srcs = [
5757
"BUILD",
5858
],
59-
visibility = ["//:__subpackages__"],
59+
visibility = ["//:__subpackages__"], # Needs skylib's root BUILD file for default_applicable_licenses
6060
)
6161

6262
# The files needed for distribution

0 commit comments

Comments
 (0)