-
Notifications
You must be signed in to change notification settings - Fork 6.8k
build: update to TypeScript 4.9 #26380
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
"@angular-devkit/build-angular": "^15.1.0-rc.0", | ||
"@angular-devkit/core": "^15.1.0-rc.0", | ||
"@angular-devkit/schematics": "^15.1.0-rc.0", | ||
"@angular/bazel": "15.1.0-next.3", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried updating @angular/bazel
as well, but I couldn't get the build to work, likely because of the changes mentioned in #26357. One of the errors was because we don't need the devmode-es2020-bazel.patch
, but after removing it, I had some very vague errors from Bazel like:
ERROR: material2/src/material/progress-bar/testing/BUILD.bazel:5:10: for src/material/progress-bar/testing/index.d.ts, previous action: action 'Compiling Angular templates (full prodmode) //src/material/progress-bar/testing:testing', attempted action: action 'Compiling Angular templates (full devmode) //src/material/progress-bar/testing:testing'
ERROR: Analysis of target '//src/material/progress-bar/testing:testing' failed; build aborted:
INFO: Elapsed time: 51.064s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (284 packages loaded, 13767 targets configured)
error Command failed with exit code 1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, it needs much more changes in this repo to be able to update. It should be fine not updating, or do you need it for TS?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't need it for the TS update, I tried updating it for consistency.
Updates most packages to the latest RC and to TypeScript 4.9.
Updates most packages to the latest RC and to TypeScript 4.9. (cherry picked from commit e337fe8)
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Updates most packages to the latest RC and to TypeScript 4.9.