Skip to content

Commit 132c00c

Browse files
alan-agius4atscott
authored andcommitted
build: force benchpress to be compiled with ES2020 (angular#49332)
Benchpress requires a decent amount of work to be ES2022 compliant. This will be done in a followup PR. PR Close angular#49332
1 parent 262c989 commit 132c00c

3 files changed

Lines changed: 10 additions & 3 deletions

File tree

packages/benchpress/BUILD.bazel

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ ts_library(
1010
"src/**/*.ts",
1111
],
1212
),
13+
# TODO(alanagius) fix benchpress to compile with es2022
14+
devmode_target = "es2020",
15+
prodmode_target = "es2020",
1316
deps = [
1417
"//packages:types",
1518
"//packages/core",

packages/benchpress/test/BUILD.bazel

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ ts_library(
44
name = "test_lib",
55
testonly = True,
66
srcs = glob(["**/*.ts"]),
7+
# TODO(alanagius) fix benchpress to compile with es2022
8+
devmode_target = "es2020",
9+
prodmode_target = "es2020",
710
deps = [
811
"//packages:types",
912
"//packages/benchpress",

tools/defaults.bzl

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,8 @@ def ts_library(
103103
deps = [],
104104
module_name = None,
105105
package_name = None,
106+
devmode_target = "es2022",
107+
prodmode_target = "es2022",
106108
**kwargs):
107109
"""Default values for ts_library"""
108110
deps = deps + ["@npm//tslib"]
@@ -122,21 +124,20 @@ def ts_library(
122124
if not package_name:
123125
package_name = _default_module_name(testonly)
124126

125-
default_target = "es2022"
126127
default_module = "esnext"
127128

128129
_ts_library(
129130
name = name,
130131
tsconfig = tsconfig,
131132
testonly = testonly,
132133
deps = deps,
133-
devmode_target = default_target,
134+
devmode_target = devmode_target,
134135
devmode_module = default_module,
135136
# For prodmode, the target is set to `ES2022`. `@bazel/typecript` sets `ES2015` by
136137
# default. Note that this should be in sync with the `ng_module` tsconfig generation.
137138
# https://github.com/bazelbuild/rules_nodejs/blob/901df3868e3ceda177d3ed181205e8456a5592ea/third_party/github.com/bazelbuild/rules_typescript/internal/common/tsconfig.bzl#L195
138139
# https://github.com/bazelbuild/rules_nodejs/blob/9b36274dba34204625579463e3da054a9f42cb47/packages/typescript/internal/build_defs.bzl#L85.
139-
prodmode_target = default_target,
140+
prodmode_target = prodmode_target,
140141
prodmode_module = default_module,
141142
# `module_name` is used for AMD module names within emitted JavaScript files.
142143
module_name = module_name,

0 commit comments

Comments
 (0)