Skip to content

Commit 3a56871

Browse files
committed
fix(bazel): relax node.js check in integration checks
This error currently breaks components.
1 parent 66a6238 commit 3a56871

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

bazel/integration/index.bzl

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,8 +117,9 @@ def _unwrap_label_keyed_mappings(ctx, dict, description):
117117
# The nodejs toolchain now provides node and npm binaries, as long as node is the first file
118118
# provided in the DefaultInfo files list, we continue as expected.
119119
if name == "node":
120-
if (len(files) != 2 or not files[0].short_path.endswith("bin/node")):
121-
fail("Expected %s target %s to be an provide two files, of which the first is `node`")
120+
# TODO: replace `> 2` with `!= 2` when migration from the old nodejs_toolchain
121+
if (len(files) > 2 or not files[0].short_path.endswith("bin/node")):
122+
fail("Expected %s target %s to be an provide up to two files, of which the first is `node`." % (description, target))
122123

123124
# For all non-node, we expect a single file or an executable which will only have one file.
124125
elif len(files) != 1:

0 commit comments

Comments
 (0)