refactor: make npm targets foo__nested_node_modules, `foo__all_file…

…s` & `foo__contents` private
bazel-contrib · Nov 26, 2019 · 39d2305 · 39d2305
1 parent 5de3c6c
commit 39d2305
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/internal/npm_install/generate_build_file.js b/internal/npm_install/generate_build_file.js
@@ -835,11 +835,13 @@ filegroup(
 
 filegroup(
     name = "${pkg._name}__nested_node_modules",${nestedNodeModulesStarlark}
+    visibility = ["//visibility:private"],
 )
 
 filegroup(
     name = "${pkg._name}__all_files",
     srcs = [":${pkg._name}__files", ":${pkg._name}__nested_node_modules"],
+    visibility = ["//:__subpackages__"],
 )
 
 node_module_library(
@@ -853,6 +855,7 @@ node_module_library(
 node_module_library(
     name = "${pkg._name}__contents",
     srcs = [":${pkg._name}__all_files"],${namedSourcesStarlark}
+    visibility = ["//:__subpackages__"],
 )
 
 # ${pkg._name}__typings is the subset of ${pkg._name}__contents that are declarations