diff --git a/site/docs/skylark/build-style.md b/site/docs/skylark/build-style.md index 30bae9a90e6675..756a450c0e6426 100644 --- a/site/docs/skylark/build-style.md +++ b/site/docs/skylark/build-style.md @@ -117,15 +117,15 @@ reference (`:x` instead of `//x`). ## Visibility -Do not set the default visibility of a package to `//visibility:public`. -`//visibility:public` should be individually set for targets in the -project's public API. These could be libraries which are designed to be -depended on by external projects or binaries that could be used by an -external project's build process. - -Otherwise, visibility should be scoped as tightly as possible, while still -allowing access by tests and reverse dependencies. Prefer using `__pkg__` to -`__subpackages__`. +Visibility should be scoped as tightly as possible, while still allowing access +by tests and reverse dependencies. Use `__pkg__` and `__subpackages__` as +appropriate. + +Avoid setting package `default_visibility` to `//visibility:public`. +`//visibility:public` should be individually set only for targets in the +project's public API. These could be libraries that are designed to be depended +on by external projects or binaries that could be used by an external project's +build process. ## Dependencies