Skip to content

Commit

Permalink
refactor: make unresolved_symlinks_enabled attribute of js_binary man…
Browse files Browse the repository at this point in the history
…datory (#1571)
  • Loading branch information
gregmagolan authored and jbedard committed May 6, 2024
1 parent 1382262 commit b9358e9
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
4 changes: 2 additions & 2 deletions docs/js_binary.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 1 addition & 2 deletions js/private/js_binary.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -267,8 +267,7 @@ _ATTRS = {
attribute based on a `config_setting` rule.
See /js/private/BUILD.bazel in rules_js for an example.
""",
# TODO(2.0): make this mandatory so that downstream binary rules that inherit these attributes are required to set it
mandatory = False,
mandatory = True,
),
"node_toolchain": attr.label(
doc = """The Node.js toolchain to use for this target.
Expand Down

0 comments on commit b9358e9

Please sign in to comment.