Skip to content

Commit 183f772

Browse files
devversionclydin
authored andcommitted
build: update to bazel 6
This is necessary for an incremental migration to `rules_js` which requires Bazel v6. Bazel v6 removed the managed directories feature, which means we no longer can rely on symlinked node modules as the Bazel repository; but rather need to duplicate dependencies. This is okay/acceptable to enable the incremental migration.
1 parent 1bf9381 commit 183f772

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

.bazelversion

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
5.4.1
1+
6.5.0

WORKSPACE

+1-7
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,4 @@
1-
workspace(
2-
name = "angular_cli",
3-
managed_directories = {"@npm": ["node_modules"]},
4-
)
1+
workspace(name = "angular_cli")
52

63
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
74

@@ -122,9 +119,6 @@ yarn_install(
122119
# 2. Incompatibilites with the `ts_library` rule.
123120
exports_directories_only = False,
124121
package_json = "//:package.json",
125-
# We prefer to symlink the `node_modules` to only maintain a single install.
126-
# See https://github.com/angular/dev-infra/pull/446#issuecomment-1059820287 for details.
127-
symlink_node_modules = True,
128122
yarn = "//:.yarn/releases/yarn-4.5.0.cjs",
129123
yarn_lock = "//:yarn.lock",
130124
)

0 commit comments

Comments
 (0)