Skip to content

Commit e7e1ec3

Browse files
devversionmmalerba
authored andcommitted
build: remove unused angular bazel postinstall patch (#18373)
In the past, when the framework repo updated their `rules_nodejs` version, a patch on the components was needed in order to make the `components-repo-unit-tests` job on framework green. Since this update PR already landed, and we updated to the Angular version that includes these changes, we can remove the patch.
1 parent 0f7a828 commit e7e1ec3

File tree

2 files changed

+0
-89
lines changed

2 files changed

+0
-89
lines changed

tools/bazel/angular_bazel_rules_nodejs_1.0.0.patch

Lines changed: 0 additions & 77 deletions
This file was deleted.

tools/bazel/postinstall-patches.js

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,18 +29,6 @@ shelljs.cd(projectDir);
2929
// Workaround for https://github.com/angular/angular/issues/18810.
3030
shelljs.exec('ngc -p angular-tsconfig.json');
3131

32-
try {
33-
// Temporary patch to make @angular/bazel compatible with rules_nodejs 1.0.0.
34-
// This is needed to resolve the dependency sandwich between angular components and
35-
// repo framework. It can be removed with a future @angular/bazel update.
36-
// try/catch needed for this the material CI tests to work in angular/repo
37-
applyPatch(path.join(__dirname, './angular_bazel_rules_nodejs_1.0.0.patch'));
38-
} catch (_) {}
39-
40-
// Temporary patch for ts-api-guardian to be compatible with rules_nodejs 1.0.0.
41-
// TODO: a new ts-api-guardian release is needed.
42-
applyPatch(path.join(__dirname, './ts_api_guardian_rules_nodejs_1.0.0.patch'));
43-
4432
// Workaround for https://github.com/angular/angular/issues/30586. It's not possible to
4533
// enable tsickle decorator processing without enabling import rewriting to closure.
4634
// This replacement allows us to enable decorator processing without rewriting imports.

0 commit comments

Comments
 (0)