docs: Clarify that $npm_package.publish requires publishable = True

This fact is already mentioned further down in the docs, but this commit makes it more apparent when only scanning the summary. The default value for `publishable` was changed in #1683.
aspect-build · Sep 30, 2024 · e00378d · e00378d
1 parent f0c67a2
commit e00378d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/docs/npm_package.md b/docs/npm_package.md
diff --git a/npm/private/npm_package.bzl b/npm/private/npm_package.bzl
@@ -155,7 +155,7 @@ def npm_package(
 
     This target can be used as the `src` attribute to `npm_link_package`.
 
-    The macro also produces a target `[name].publish`, that can be run to publish to an npm registry.
+    With `publishable = True` the macro also produces a target `[name].publish`, that can be run to publish to an npm registry.
     Under the hood, this target runs `npm publish`. You can pass arguments to npm by escaping them from Bazel using a double-hyphen,
     for example: `bazel run //path/to:my_package.publish -- --tag=next`