Skip to content

Commit caf37a9

Browse files
devversionandrewseguin
authored andcommitted
build: remove unnecessary postinstall rules_nodejs patch (#22727)
We recently updated to `rules_nodejs` 3.5.0. This contained a fix for something we recently tried to achieve using a postinstall patch. The PR that updated to 3.5.0 became a little out-of-date due to the PR introducing the postinstall patch being merged, so that the failure only surfaced when the update PR got merged. (cherry picked from commit 9d87461)
1 parent 365ed7c commit caf37a9

File tree

2 files changed

+0
-41
lines changed

2 files changed

+0
-41
lines changed

tools/postinstall/apply-patches.js

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,6 @@ function applyPatches() {
5555
// Workaround for https://github.com/angular/angular/issues/18810.
5656
shelljs.exec('ngc -p angular-tsconfig.json');
5757

58-
// TODO(devversion): Remove once https://github.com/bazelbuild/rules_nodejs/commit/4965db6b775d99c370fe0fad7582168d55d520ec is available.
59-
applyPatch(path.join(__dirname, 'bazel-protractor-server-configuration.patch'));
60-
6158
// Workaround for: https://github.com/angular/angular/issues/32651. We just do not
6259
// generate re-exports for secondary entry-points. Similar to what "ng-packagr" does.
6360
searchAndReplace(

tools/postinstall/bazel-protractor-server-configuration.patch

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

0 commit comments

Comments
 (0)