Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 5802f80

Browse files
josephperrottcrisbeto
authored andcommitted
build: no longer rely on archive packages for integration tests (#62996)
Now that the packages are avaiable at the base node_modules we can treat local packages as normal packages and not leverage the archive package versions PR Close #62996
1 parent 648bbb0 commit 5802f80

File tree

7 files changed

+37
-6
lines changed

7 files changed

+37
-6
lines changed

‎.bazelignore‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ packages/core/node_modules
5555
packages/core/test/bundling/node_modules
5656
packages/elements/node_modules
5757
packages/forms/node_modules
58+
packages/language-service/node_modules
5859
packages/platform-browser/node_modules
5960
packages/platform-server/node_modules
6061
packages/platform-browser-dynamic/node_modules

‎WORKSPACE‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,7 @@ npm_translate_lock(
9292
"//packages/core/test/bundling:package.json",
9393
"//packages/elements:package.json",
9494
"//packages/forms:package.json",
95+
"//packages/language-service:package.json",
9596
"//packages/localize:package.json",
9697
"//packages/platform-browser:package.json",
9798
"//packages/platform-browser-dynamic:package.json",
@@ -218,7 +219,7 @@ http_archive(
218219

219220
git_repository(
220221
name = "devinfra",
221-
commit = "4d2f875ec29ee71e0fe1a349a99c5ab2ccb71e30",
222+
commit = "73442df90c353b4b777574ffac1ede88ee89ac4d",
222223
remote = "https://github.com/angular/dev-infra.git",
223224
)
224225

‎integration/index.bzl‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
load("@devinfra//bazel/integration:index.bzl", "integration_test")
99
load("//:packages.bzl", "INTEGRATION_PACKAGES")
1010

11-
NPM_PACKAGE_ARCHIVES = [
11+
NPM_PACKAGE_ARCHIVES = INTEGRATION_PACKAGES + [
1212
"@babel/core",
1313
"@rollup/plugin-babel",
1414
"@rollup/plugin-node-resolve",
@@ -65,9 +65,6 @@ def _ng_integration_test(name, setup_chromium = False, **kwargs):
6565
for pkg in NPM_PACKAGE_ARCHIVES:
6666
if pkg not in pinned_npm_packages:
6767
npm_packages["//:node_modules/%s/dir" % pkg] = pkg
68-
for pkg in INTEGRATION_PACKAGES:
69-
last_segment_name = pkg.split("/")[-1]
70-
npm_packages["//packages/%s:npm_package_archive" % last_segment_name] = pkg
7168

7269
integration_test(
7370
name = name,
@@ -87,6 +84,7 @@ def _ng_integration_test(name, setup_chromium = False, **kwargs):
8784
environment = environment,
8885
toolchains = toolchains,
8986
tool_mappings = {
87+
"@pnpm//:pnpm": "pnpm",
9088
"//:yarn_vendored": "yarn",
9189
"@nodejs_toolchains//:resolved_toolchain": "node",
9290
},

‎package.json‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
"@angular-devkit/core": "20.2.0-next.2",
5353
"@angular-devkit/schematics": "20.2.0-next.2",
5454
"@angular/animations": "workspace:*",
55+
"@angular/benchpress": "workspace: *",
5556
"@angular/build": "20.2.0-next.2",
5657
"@angular/cdk": "20.2.0-next.2",
5758
"@angular/cli": "20.2.0-next.2",
@@ -61,13 +62,16 @@
6162
"@angular/core": "workspace:*",
6263
"@angular/elements": "workspace:*",
6364
"@angular/forms": "workspace:*",
65+
"@angular/language-service": "workspace: *",
66+
"@angular/localize": "workspace: *",
6467
"@angular/material": "20.2.0-next.2",
6568
"@angular/platform-browser": "workspace:*",
6669
"@angular/platform-browser-dynamic": "workspace:*",
6770
"@angular/platform-server": "workspace:*",
6871
"@angular/router": "workspace:*",
6972
"@angular/service-worker": "workspace:*",
7073
"@angular/ssr": "20.2.0-next.2",
74+
"@angular/upgrade": "workspace: *",
7175
"@babel/cli": "7.28.0",
7276
"@babel/core": "7.28.0",
7377
"@babel/generator": "7.28.0",

‎packages/language-service/BUILD.bazel‎

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
load("@aspect_rules_esbuild//esbuild:defs.bzl", "esbuild")
22
load("//tools:defaults.bzl", "pkg_npm")
3-
load("//tools:defaults2.bzl", "extract_types", "ts_config", "ts_project")
3+
load("//tools:defaults2.bzl", "extract_types", "npm_package", "ts_config", "ts_project")
44

55
package(default_visibility = ["//visibility:public"])
66

@@ -79,3 +79,11 @@ pkg_npm(
7979
"//packages/language-service/bundles:language-service.js",
8080
],
8181
)
82+
83+
npm_package(
84+
name = "pkg",
85+
srcs = [":npm_package"],
86+
replace_prefixes = {
87+
"npm_package/": "",
88+
},
89+
)

‎pnpm-lock.yaml‎

Lines changed: 14 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎pnpm-workspace.yaml‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ packages:
1313
- packages/core/
1414
- packages/core/test/bundling
1515
- packages/forms/
16+
- packages/language-service/
1617
- packages/platform-browser/
1718
- packages/platform-server/
1819
- packages/platform-browser-dynamic/
@@ -43,3 +44,7 @@ overrides:
4344
'@angular/forms': 'workspace:*'
4445
'@angular/elements': 'workspace:*'
4546
'@angular/animations': 'workspace:*'
47+
'@angular/benchpress': 'workspace: *'
48+
'@angular/language-service': 'workspace: *'
49+
'@angular/localize': 'workspace: *'
50+
'@angular/upgrade': 'workspace: *'

0 commit comments

Comments
 (0)