From b027899cfb1a220026c511767bf3e7877a749369 Mon Sep 17 00:00:00 2001 From: Alan Agius <17563226+alan-agius4@users.noreply.github.com> Date: Thu, 14 Aug 2025 11:30:30 +0000 Subject: [PATCH] fix(bazel): relax node.js check in integration checks This error currently breaks components. --- bazel/integration/index.bzl | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/bazel/integration/index.bzl b/bazel/integration/index.bzl index 84a002ccb..8457d053b 100644 --- a/bazel/integration/index.bzl +++ b/bazel/integration/index.bzl @@ -117,8 +117,9 @@ def _unwrap_label_keyed_mappings(ctx, dict, description): # The nodejs toolchain now provides node and npm binaries, as long as node is the first file # provided in the DefaultInfo files list, we continue as expected. if name == "node": - if (len(files) != 2 or not files[0].short_path.endswith("bin/node")): - fail("Expected %s target %s to be an provide two files, of which the first is `node`") + # TODO: replace `> 2` with `!= 2` when migration from the old nodejs_toolchain + if (len(files) > 2 or not files[0].short_path.endswith("bin/node")): + fail("Expected %s target %s to be an provide up to two files, of which the first is `node`." % (description, target)) # For all non-node, we expect a single file or an executable which will only have one file. elif len(files) != 1: