Skip to content

Commit 0e40b8c

Browse files
devversionmmalerba
authored andcommitted
build: do not hide output of bazel yarn install (#18765)
By default, the `yarn_install` output is hidden. We should enable it so that we can see progress, or potential postinstall errors.
1 parent d318b27 commit 0e40b8c

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

WORKSPACE

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,13 +49,18 @@ node_repositories(
4949

5050
yarn_install(
5151
name = "npm",
52+
# Redirects Yarn `stdout` output to `stderr`. This ensures that stdout is not accidentally
53+
# polluted when Bazel runs Yarn. Workaround until the upstream fix is available:
54+
# https://github.com/bazelbuild/bazel/pull/10611.
55+
args = ["1>&2"],
5256
# We add the postinstall patches file, and ngcc main fields update script here so
5357
# that Yarn will rerun whenever one of these files has been modified.
5458
data = [
5559
"//:tools/postinstall/apply-patches.js",
5660
"//:tools/postinstall/update-ngcc-main-fields.js",
5761
],
5862
package_json = "//:package.json",
63+
quiet = False,
5964
yarn_lock = "//:yarn.lock",
6065
)
6166

0 commit comments

Comments
 (0)