Skip to content

Commit 093d007

Browse files
devversionmmalerba
authored andcommitted
build: exclude devserver targets from ci test jobs (#17990)
We don't want to build the runfile tree's for the executable devserver targets. In the CI, we are only interested in building the actual "ng_module" target to ensure the dev-app/e2e-app are not broken. Building the devserver runfiles is just wasteful.
1 parent 1fcade2 commit 093d007

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

src/dev-app/BUILD.bazel

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,7 @@ dev_server(
177177
# artifact output as workspace root.
178178
"angular_material",
179179
],
180+
tags = ["manual"],
180181
deps = [
181182
":dev-app",
182183
],

src/e2e-app/BUILD.bazel

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -90,17 +90,14 @@ sass_binary(
9090
ts_devserver(
9191
name = "devserver",
9292
testonly = True,
93-
# Root paths can be used simplify the loading of files from external Bazel repositories
94-
# (such as the Bazel managed deps repository called "npm")
9593
additional_root_paths = [
9694
"npm/node_modules",
9795
],
9896
# Name of the AMD module that should be required on page load.
9997
entry_module = "angular_material/src/e2e-app/main",
10098
port = 4200,
101-
# Scripts which will be included in the serving_path bundle after "require.js" has been
102-
# loaded.
103-
# TODO(jelbourn): remove UMDs from here once we don't have to manually include them
99+
# Scripts which will be included in the serving_path bundle after
100+
# RequireJS has been loaded.
104101
scripts = [
105102
":devserver-configure.js",
106103
"//tools/rxjs:rxjs_umd_modules",
@@ -120,7 +117,6 @@ ts_devserver(
120117
":index.html",
121118
":theme",
122119
],
123-
# Dependencies that produce JavaScript output will be automatically included in the
124-
# serving_path bundle
120+
tags = ["manual"],
125121
deps = [":e2e-app"],
126122
)

0 commit comments

Comments
 (0)