Skip to content

Commit

Permalink
feat(builtin): flip the default of the strict_visibility flag on the …
Browse files Browse the repository at this point in the history
…npm and yarn install rules to True
  • Loading branch information
mattem committed Oct 18, 2020
1 parent 8bbc27b commit 829f068
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions internal/npm_install/npm_install.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -84,14 +84,12 @@ fine grained npm dependencies.
doc = "If stdout and stderr should be printed to the terminal.",
),
"strict_visibility": attr.bool(
default = False,
default = True,
doc = """Turn on stricter visibility for generated BUILD.bazel files
When enabled, only dependencies within the given `package.json` file are given public visibility.
All transitive dependencies are given limited visibility, enforcing that all direct dependencies are
listed in the `package.json` file.
Currently the default is set `False`, but will likely be flipped `True` in rules_nodejs 3.0.0
""",
),
"symlink_node_modules": attr.bool(
Expand Down

0 comments on commit 829f068

Please sign in to comment.