Skip to content

Commit

Permalink
build: remove unused managed directory bazel setup for node modules
Browse files Browse the repository at this point in the history
As of RNJ v5, the node modules are no longer symlinked by default. Rather
a new separate install is maintained inside Bazel. This was a little
controversial in the past since it made debugging harder, and required
more IO locally/or even downloads depending on how Yarn is cached/configured
locally.

With the current setup, symlinks are no longer enabled and we can remove
the managed directory configuration.

bazel-contrib/rules_nodejs#3214
  • Loading branch information
devversion committed Mar 2, 2022
1 parent 583f8b7 commit 58ae198
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion WORKSPACE
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
workspace(
name = "dev-infra",
managed_directories = {"@npm": ["node_modules"]},
)

load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive", "http_file")
Expand Down

0 comments on commit 58ae198

Please sign in to comment.