From 6161c50214f2bb882c72452f6bda387a80f7e192 Mon Sep 17 00:00:00 2001 From: Alan Agius Date: Thu, 16 Mar 2023 13:19:54 +0000 Subject: [PATCH] build: force benchpress to be compiled with ES2020 (#49559) Benchpress requires a decent amount of work to be ES2022 compliant. This will be done in a followup PR. PR Close #49559 --- packages/benchpress/BUILD.bazel | 3 +++ packages/benchpress/test/BUILD.bazel | 3 +++ tools/defaults.bzl | 7 ++++--- 3 files changed, 10 insertions(+), 3 deletions(-) diff --git a/packages/benchpress/BUILD.bazel b/packages/benchpress/BUILD.bazel index dd207fbce4e..9378ac34eed 100644 --- a/packages/benchpress/BUILD.bazel +++ b/packages/benchpress/BUILD.bazel @@ -10,6 +10,9 @@ ts_library( "src/**/*.ts", ], ), + # TODO(alanagius) fix benchpress to compile with es2022 + devmode_target = "es2020", + prodmode_target = "es2020", deps = [ "//packages:types", "//packages/core", diff --git a/packages/benchpress/test/BUILD.bazel b/packages/benchpress/test/BUILD.bazel index 7449f4f9d6b..0642530a1a7 100644 --- a/packages/benchpress/test/BUILD.bazel +++ b/packages/benchpress/test/BUILD.bazel @@ -4,6 +4,9 @@ ts_library( name = "test_lib", testonly = True, srcs = glob(["**/*.ts"]), + # TODO(alanagius) fix benchpress to compile with es2022 + devmode_target = "es2020", + prodmode_target = "es2020", deps = [ "//packages:types", "//packages/benchpress", diff --git a/tools/defaults.bzl b/tools/defaults.bzl index ddf67325634..60e3d460aa3 100644 --- a/tools/defaults.bzl +++ b/tools/defaults.bzl @@ -103,6 +103,8 @@ def ts_library( deps = [], module_name = None, package_name = None, + devmode_target = "es2022", + prodmode_target = "es2022", **kwargs): """Default values for ts_library""" deps = deps + ["@npm//tslib"] @@ -122,7 +124,6 @@ def ts_library( if not package_name: package_name = _default_module_name(testonly) - default_target = "es2022" default_module = "esnext" _ts_library( @@ -130,13 +131,13 @@ def ts_library( tsconfig = tsconfig, testonly = testonly, deps = deps, - devmode_target = default_target, + devmode_target = devmode_target, devmode_module = default_module, # For prodmode, the target is set to `ES2022`. `@bazel/typecript` sets `ES2015` by # default. Note that this should be in sync with the `ng_module` tsconfig generation. # https://github.com/bazelbuild/rules_nodejs/blob/901df3868e3ceda177d3ed181205e8456a5592ea/third_party/github.com/bazelbuild/rules_typescript/internal/common/tsconfig.bzl#L195 # https://github.com/bazelbuild/rules_nodejs/blob/9b36274dba34204625579463e3da054a9f42cb47/packages/typescript/internal/build_defs.bzl#L85. - prodmode_target = default_target, + prodmode_target = prodmode_target, prodmode_module = default_module, # `module_name` is used for AMD module names within emitted JavaScript files. module_name = module_name,