Skip to content

Commit

Permalink
chore: runfiles is now optional with rules_js
Browse files Browse the repository at this point in the history
  • Loading branch information
gregmagolan committed Jan 8, 2024
1 parent b9ae2ef commit 5a6dc30
Show file tree
Hide file tree
Showing 10 changed files with 8 additions and 15 deletions.
2 changes: 0 additions & 2 deletions .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@
# Take care to document any settings that you expect users to apply.
# Settings that apply only to CI are in .github/workflows/ci.bazelrc

# For rules_js
build --enable_runfiles
# Config to enable building and testing with bzlmod
common:bzlmod --enable_bzlmod --@aspect_bazel_lib//lib:flag_bzlmod

Expand Down
2 changes: 1 addition & 1 deletion MODULE.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ module(
)

bazel_dep(name = "aspect_bazel_lib", version = "1.39.1")
bazel_dep(name = "aspect_rules_js", version = "1.34.1")
bazel_dep(name = "aspect_rules_js", version = "1.35.0")
bazel_dep(name = "bazel_skylib", version = "1.5.0")
bazel_dep(name = "platforms", version = "0.0.8")
bazel_dep(name = "rules_nodejs", version = "5.8.3")
Expand Down
1 change: 0 additions & 1 deletion e2e/bundle/.bazelrc
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
build --enable_runfiles
common:bzlmod --enable_bzlmod
1 change: 0 additions & 1 deletion e2e/npm-links/.bazelrc
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
build --enable_runfiles
common:bzlmod --enable_bzlmod
2 changes: 1 addition & 1 deletion e2e/npm-links/MODULE.bazel
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
"Bazel dependencies"

bazel_dep(name = "aspect_rules_esbuild", version = "0.0.0", dev_dependency = True)
bazel_dep(name = "aspect_rules_js", version = "1.34.1", dev_dependency = True)
bazel_dep(name = "aspect_rules_js", version = "1.35.0", dev_dependency = True)
bazel_dep(name = "bazel_skylib", version = "1.5.0", dev_dependency = True)

local_path_override(
Expand Down
1 change: 0 additions & 1 deletion e2e/smoke/.bazelrc
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
build --enable_runfiles
common:bzlmod --enable_bzlmod
1 change: 0 additions & 1 deletion e2e/sourcemaps/.bazelrc
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
build --enable_runfiles
common:bzlmod --enable_bzlmod
1 change: 0 additions & 1 deletion e2e/tsconfig/.bazelrc
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
build --enable_runfiles
common:bzlmod --enable_bzlmod
6 changes: 3 additions & 3 deletions esbuild/dependencies.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -21,9 +21,9 @@ def rules_esbuild_dependencies():

http_archive(
name = "aspect_rules_js",
sha256 = "76a04ef2120ee00231d85d1ff012ede23963733339ad8db81f590791a031f643",
strip_prefix = "rules_js-1.34.1",
url = "https://github.com/aspect-build/rules_js/releases/download/v1.34.1/rules_js-v1.34.1.tar.gz",
sha256 = "a2f941e27f02e84521c2d47fd530c66d57dd6d6e44b4a4f1496fe304851d8e48",
strip_prefix = "rules_js-1.35.0",
url = "https://github.com/aspect-build/rules_js/releases/download/v1.35.0/rules_js-v1.35.0.tar.gz",
)

http_archive(
Expand Down
6 changes: 3 additions & 3 deletions internal_deps.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ def rules_esbuild_internal_deps():
# opt-in to 2.0 without forcing users to do so
http_archive(
name = "aspect_bazel_lib",
sha256 = "fc8bd670380eaba5314769abbe9fee21d641e3da06d9d26b8073a301f6d62332",
strip_prefix = "bazel-lib-2.1.0",
url = "https://github.com/aspect-build/bazel-lib/releases/download/v2.1.0/bazel-lib-v2.1.0.tar.gz",
sha256 = "a9ea6902c860918bd6928114efbc6ea2093df006af66459c9ac1637f9dd08f6a",
strip_prefix = "bazel-lib-2.1.2",
url = "https://github.com/aspect-build/bazel-lib/releases/download/v2.1.2/bazel-lib-v2.1.2.tar.gz",
)

http_archive(
Expand Down

0 comments on commit 5a6dc30

Please sign in to comment.