Skip to content

Commit 087ab46

Browse files
devversionalan-agius4
authored andcommitted
build: update dev-infra packages and account for build-tooling split from ng-dev
The dev-infra build tooling is now decoupled from `ng-dev`. This will make it easier to update `ng-dev` without necessarily needing to upgrade the whole build system, Bazel etc. This is useful when e.g. new release tool features have been added and should also be ported to active LTS branches.
1 parent 9b75c7c commit 087ab46

File tree

18 files changed

+63
-56
lines changed

18 files changed

+63
-56
lines changed

.bazelrc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,8 +118,8 @@ build:remote --jobs=150
118118

119119
# Setup the toolchain and platform for the remote build execution. The platform
120120
# is provided by the shared dev-infra package and targets k8 remote containers.
121-
build:remote --crosstool_top=@npm//@angular/dev-infra-private/bazel/remote-execution/cpp:cc_toolchain_suite
122-
build:remote --extra_toolchains=@npm//@angular/dev-infra-private/bazel/remote-execution/cpp:cc_toolchain
121+
build:remote --crosstool_top=@npm//@angular/build-tooling/bazel/remote-execution/cpp:cc_toolchain_suite
122+
build:remote --extra_toolchains=@npm//@angular/build-tooling/bazel/remote-execution/cpp:cc_toolchain
123123
build:remote --extra_execution_platforms=//tools:rbe_platform_with_network_access
124124
build:remote --host_platform=//tools:rbe_platform_with_network_access
125125
build:remote --platforms=//tools:rbe_platform_with_network_access

.ng-dev/caretaker.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { CaretakerConfig } from '@angular/dev-infra-private/ng-dev';
1+
import { CaretakerConfig } from '@angular/ng-dev';
22

33
/** The configuration for `ng-dev caretaker` commands. */
44
export const caretaker: CaretakerConfig = {

.ng-dev/commit-message.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { CommitMessageConfig } from '@angular/dev-infra-private/ng-dev';
1+
import { CommitMessageConfig } from '@angular/ng-dev';
22
import packages from '../lib/packages.js';
33

44
/**

.ng-dev/format.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { FormatConfig } from '@angular/dev-infra-private/ng-dev';
1+
import { FormatConfig } from '@angular/ng-dev';
22

33
/**
44
* Configuration for the `ng-dev format` command.

.ng-dev/github.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { GithubConfig } from '@angular/dev-infra-private/ng-dev';
1+
import { GithubConfig } from '@angular/ng-dev';
22

33
/**
44
* Github configuration for the ng-dev command. This repository is

.ng-dev/pull-request.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { PullRequestConfig } from '@angular/dev-infra-private/ng-dev';
1+
import { PullRequestConfig } from '@angular/ng-dev';
22

33
/**
44
* Configuration for the merge tool in `ng-dev`. This sets up the labels which

.ng-dev/release.mts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import '../lib/bootstrap-local.js';
22

3-
import { ReleaseConfig } from '@angular/dev-infra-private/ng-dev';
3+
import { ReleaseConfig } from '@angular/ng-dev';
44
import packages from '../lib/packages.js';
55
import buildPackages from '../scripts/build.js';
66

package.json

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@
2424
"build:bazel": "node ./bin/devkit-admin build-bazel",
2525
"build-tsc": "tsc -p tsconfig.json",
2626
"lint": "eslint --cache --max-warnings=0 \"**/*.ts\"",
27-
"ng-dev": "cross-env TS_NODE_PROJECT=$PWD/.ng-dev/tsconfig.json TS_NODE_TRANSPILE_ONLY=1 node --no-warnings --loader ts-node/esm node_modules/@angular/dev-infra-private/ng-dev/bundles/cli.mjs",
27+
"ng-dev": "cross-env TS_NODE_PROJECT=$PWD/.ng-dev/tsconfig.json TS_NODE_TRANSPILE_ONLY=1 node --no-warnings --loader ts-node/esm node_modules/@angular/ng-dev/bundles/cli.mjs",
2828
"templates": "node ./bin/devkit-admin templates",
2929
"validate": "node ./bin/devkit-admin validate",
3030
"postinstall": "yarn webdriver-update && yarn husky install",
@@ -66,15 +66,16 @@
6666
"devDependencies": {
6767
"@ampproject/remapping": "2.2.0",
6868
"@angular/animations": "14.1.0",
69+
"@angular/build-tooling": "https://github.com/angular/dev-infra-private-build-tooling-builds.git#1addc303bef0b6acc0dca0961e9e642629f3a5cd",
6970
"@angular/cdk": "14.1.0",
7071
"@angular/common": "14.1.0",
7172
"@angular/compiler": "14.1.0",
7273
"@angular/compiler-cli": "14.1.0",
7374
"@angular/core": "14.1.0",
74-
"@angular/dev-infra-private": "https://github.com/angular/dev-infra-private-builds.git#b2da73b3dddddd6a253ee8ea48ef387b20f5aedf",
7575
"@angular/forms": "14.1.0",
7676
"@angular/localize": "14.1.0",
7777
"@angular/material": "14.1.0",
78+
"@angular/ng-dev": "https://github.com/angular/dev-infra-private-ng-dev-builds.git#865c7687cdca2bd512040330e1677eecaa26e46a",
7879
"@angular/platform-browser": "14.1.0",
7980
"@angular/platform-browser-dynamic": "14.1.0",
8081
"@angular/platform-server": "14.1.0",

packages/angular_devkit/architect/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ load("//tools:defaults.bzl", "pkg_npm", "ts_library")
88

99
# @external_begin
1010
load("//tools:ts_json_schema.bzl", "ts_json_schema")
11-
load("@npm//@angular/dev-infra-private/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
11+
load("@npm//@angular/build-tooling/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
1212
# @external_end
1313

1414
licenses(["notice"]) # MIT

packages/angular_devkit/build_angular/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
load("@npm//@bazel/jasmine:index.bzl", "jasmine_node_test")
77
load("//tools:defaults.bzl", "pkg_npm", "ts_library")
88
load("//tools:ts_json_schema.bzl", "ts_json_schema")
9-
load("@npm//@angular/dev-infra-private/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
9+
load("@npm//@angular/build-tooling/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
1010

1111
licenses(["notice"]) # MIT
1212

packages/angular_devkit/build_webpack/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
load("@npm//@bazel/jasmine:index.bzl", "jasmine_node_test")
77
load("//tools:defaults.bzl", "pkg_npm", "ts_library")
88
load("//tools:ts_json_schema.bzl", "ts_json_schema")
9-
load("@npm//@angular/dev-infra-private/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
9+
load("@npm//@angular/build-tooling/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
1010

1111
licenses(["notice"]) # MIT
1212

packages/angular_devkit/core/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ load("@npm//@bazel/jasmine:index.bzl", "jasmine_node_test")
22
load("//tools:defaults.bzl", "pkg_npm", "ts_library")
33

44
# @external_begin
5-
load("@npm//@angular/dev-infra-private/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
5+
load("@npm//@angular/build-tooling/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
66
# @external_end
77

88
# Copyright Google Inc. All Rights Reserved.

packages/angular_devkit/schematics/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ load("@npm//@bazel/jasmine:index.bzl", "jasmine_node_test")
22
load("//tools:defaults.bzl", "pkg_npm", "ts_library")
33

44
# @external_begin
5-
load("@npm//@angular/dev-infra-private/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
5+
load("@npm//@angular/build-tooling/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
66
# @external_end
77

88
# Copyright Google Inc. All Rights Reserved.

packages/ngtools/webpack/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
load("@npm//@bazel/jasmine:index.bzl", "jasmine_node_test")
77
load("//tools:defaults.bzl", "pkg_npm", "ts_library")
8-
load("@npm//@angular/dev-infra-private/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
8+
load("@npm//@angular/build-tooling/bazel/api-golden:index.bzl", "api_golden_test_npm_package")
99

1010
licenses(["notice"]) # MIT
1111

scripts/templates.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ async function _runTemplate(inputPath: string, outputPath: string, logger: loggi
1818

1919
// TODO(ESM): Consider making this an actual import statement.
2020
const { COMMIT_TYPES, ScopeRequirement } = await new Function(
21-
`return import('@angular/dev-infra-private/ng-dev');`,
21+
`return import('@angular/ng-dev');`,
2222
)();
2323

2424
const template = require(inputPath).default;

tools/BUILD.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ platform(
3434
exec_properties = {
3535
"dockerNetwork": "standard",
3636
},
37-
parents = ["@npm//@angular/dev-infra-private/bazel/remote-execution:platform"],
37+
parents = ["@npm//@angular/build-tooling/bazel/remote-execution:platform"],
3838
)
3939

4040
# @external_end

tools/defaults.bzl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
load("@npm//@bazel/concatjs/internal:build_defs.bzl", _ts_library = "ts_library_macro")
44
load("@build_bazel_rules_nodejs//:index.bzl", "copy_to_bin", _js_library = "js_library", _pkg_npm = "pkg_npm")
55
load("@rules_pkg//:pkg.bzl", "pkg_tar")
6-
load("@npm//@angular/dev-infra-private/bazel:extract_js_module_output.bzl", "extract_js_module_output")
6+
load("@npm//@angular/build-tooling/bazel:extract_js_module_output.bzl", "extract_js_module_output")
77
load("@aspect_bazel_lib//lib:utils.bzl", "to_label")
88
load("@aspect_bazel_lib//lib:jq.bzl", "jq")
99
load("@aspect_bazel_lib//lib:copy_to_directory.bzl", "copy_to_directory")

yarn.lock

Lines changed: 43 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,42 @@
122122
"@angular/core" "^13.0.0 || ^14.0.0-0"
123123
reflect-metadata "^0.1.13"
124124

125+
"@angular/build-tooling@https://github.com/angular/dev-infra-private-build-tooling-builds.git#1addc303bef0b6acc0dca0961e9e642629f3a5cd":
126+
version "0.0.0-fa61d03a603e04af2b66f3598f1af01da1e1cfb1"
127+
resolved "https://github.com/angular/dev-infra-private-build-tooling-builds.git#1addc303bef0b6acc0dca0961e9e642629f3a5cd"
128+
dependencies:
129+
"@angular-devkit/build-angular" "14.1.0-rc.3"
130+
"@angular/benchpress" "0.3.0"
131+
"@babel/core" "^7.16.0"
132+
"@bazel/buildifier" "5.1.0"
133+
"@bazel/concatjs" "5.5.2"
134+
"@bazel/esbuild" "5.5.2"
135+
"@bazel/protractor" "5.5.2"
136+
"@bazel/runfiles" "5.5.2"
137+
"@bazel/terser" "5.5.2"
138+
"@bazel/typescript" "5.5.2"
139+
"@microsoft/api-extractor" "7.28.4"
140+
"@types/browser-sync" "^2.26.3"
141+
"@types/node" "16.10.9"
142+
"@types/selenium-webdriver" "^4.0.18"
143+
"@types/send" "^0.17.1"
144+
"@types/tmp" "^0.2.1"
145+
"@types/uuid" "^8.3.1"
146+
"@types/yargs" "^17.0.0"
147+
browser-sync "^2.27.7"
148+
clang-format "1.8.0"
149+
prettier "2.7.1"
150+
protractor "^7.0.0"
151+
selenium-webdriver "4.3.1"
152+
send "^0.18.0"
153+
source-map "^0.7.4"
154+
tmp "^0.2.1"
155+
"true-case-path" "^2.2.1"
156+
tslib "^2.3.0"
157+
typescript "~4.7.3"
158+
uuid "^8.3.2"
159+
yargs "^17.0.0"
160+
125161
"@angular/cdk@14.1.0":
126162
version "14.1.0"
127163
resolved "https://registry.yarnpkg.com/@angular/cdk/-/cdk-14.1.0.tgz#abf5ad3cdd96e42666bce949833ac1c2405ab031"
@@ -168,43 +204,6 @@
168204
dependencies:
169205
tslib "^2.3.0"
170206

171-
"@angular/dev-infra-private@https://github.com/angular/dev-infra-private-builds.git#b2da73b3dddddd6a253ee8ea48ef387b20f5aedf":
172-
version "0.0.0-114c5a9e0c063e65dc42ded4d2ae07a3cca5418a"
173-
resolved "https://github.com/angular/dev-infra-private-builds.git#b2da73b3dddddd6a253ee8ea48ef387b20f5aedf"
174-
dependencies:
175-
"@angular-devkit/build-angular" "14.1.0-rc.3"
176-
"@angular/benchpress" "0.3.0"
177-
"@babel/core" "^7.16.0"
178-
"@bazel/buildifier" "5.1.0"
179-
"@bazel/concatjs" "5.5.2"
180-
"@bazel/esbuild" "5.5.2"
181-
"@bazel/protractor" "5.5.2"
182-
"@bazel/runfiles" "5.5.2"
183-
"@bazel/terser" "5.5.2"
184-
"@bazel/typescript" "5.5.2"
185-
"@microsoft/api-extractor" "7.28.4"
186-
"@types/browser-sync" "^2.26.3"
187-
"@types/node" "16.10.9"
188-
"@types/selenium-webdriver" "^4.0.18"
189-
"@types/send" "^0.17.1"
190-
"@types/tmp" "^0.2.1"
191-
"@types/uuid" "^8.3.1"
192-
"@types/yargs" "^17.0.0"
193-
"@yarnpkg/lockfile" "^1.1.0"
194-
browser-sync "^2.27.7"
195-
clang-format "1.8.0"
196-
prettier "2.7.1"
197-
protractor "^7.0.0"
198-
selenium-webdriver "4.3.1"
199-
send "^0.18.0"
200-
source-map "^0.7.4"
201-
tmp "^0.2.1"
202-
"true-case-path" "^2.2.1"
203-
tslib "^2.3.0"
204-
typescript "~4.7.3"
205-
uuid "^8.3.2"
206-
yargs "^17.0.0"
207-
208207
"@angular/forms@14.1.0":
209208
version "14.1.0"
210209
resolved "https://registry.yarnpkg.com/@angular/forms/-/forms-14.1.0.tgz#17a7ec211a11b75572d89359f08c277cfcdb1210"
@@ -228,6 +227,13 @@
228227
dependencies:
229228
tslib "^2.3.0"
230229

230+
"@angular/ng-dev@https://github.com/angular/dev-infra-private-ng-dev-builds.git#865c7687cdca2bd512040330e1677eecaa26e46a":
231+
version "0.0.0-fa61d03a603e04af2b66f3598f1af01da1e1cfb1"
232+
resolved "https://github.com/angular/dev-infra-private-ng-dev-builds.git#865c7687cdca2bd512040330e1677eecaa26e46a"
233+
dependencies:
234+
"@yarnpkg/lockfile" "^1.1.0"
235+
typescript "~4.7.3"
236+
231237
"@angular/platform-browser-dynamic@14.1.0":
232238
version "14.1.0"
233239
resolved "https://registry.yarnpkg.com/@angular/platform-browser-dynamic/-/platform-browser-dynamic-14.1.0.tgz#9e30c5c4d2ff507402dcb36a5878068d8d900b81"

0 commit comments

Comments
 (0)