Skip to content

Commit 83426fa

Browse files
authored
Merge pull request #10328 from hzeller/feature-20260504-js-dev
Move JS dependencies in the dev-dependency section.
2 parents 3b64247 + 25f2816 commit 83426fa

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

MODULE.bazel

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -70,10 +70,6 @@ bazel_dep(name = "tcmalloc", version = "0.0.0-20250927-12f2552")
7070
bazel_dep(name = "yaml-cpp", version = "0.9.0")
7171
bazel_dep(name = "zlib", version = "1.3.1.bcr.8")
7272

73-
# JavaScript / web UI — only used for JS unit tests in src/web/test
74-
bazel_dep(name = "aspect_rules_js", version = "3.0.2", dev_dependency = True)
75-
bazel_dep(name = "rules_nodejs", version = "6.7.3", dev_dependency = True)
76-
7773
# A from source build of QT that allows it to link into OpenROAD.
7874
# Building like any other bazel project. scripts in the docker folder
7975
# of this project generate stubs .ifso for things like X11 that will
@@ -97,6 +93,12 @@ bazel_dep(name = "buildifier_prebuilt", version = "8.5.1.2", dev_dependency = Tr
9793
bazel_dep(name = "googletest", version = "1.17.0.bcr.2", dev_dependency = True)
9894
bazel_dep(name = "rules_verilator", version = "0.1.0", dev_dependency = True)
9995
bazel_dep(name = "verilator", version = "5.036.bcr.3", dev_dependency = True)
96+
97+
# JavaScript / web UI — only used for JS unit tests in src/web/test
98+
bazel_dep(name = "aspect_rules_js", version = "3.0.2", dev_dependency = True)
99+
bazel_dep(name = "rules_nodejs", version = "6.7.3", dev_dependency = True)
100+
101+
# ORFS
100102
bazel_dep(name = "bazel-orfs", dev_dependency = True)
101103
bazel_dep(name = "bazel-orfs-verilog", dev_dependency = True)
102104

@@ -131,7 +133,7 @@ llvm.toolchain(
131133
use_repo(llvm, "llvm_toolchain")
132134
# use_repo(llvm, "llvm_toolchain_llvm") # if you depend on specific tools in scripts
133135

134-
# FYI: Comment out on NixOS where you'd like to use the local clang toolchain.
136+
# FYI: Comment out @llvm_toolchain//:all on NixOS and use the local clang toolchain instead.
135137
register_toolchains(
136138
"@llvm_toolchain//:all",
137139
"@rules_verilator//verilator:verilator_toolchain",

0 commit comments

Comments
 (0)