-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
build: bump Angular dependencies and drop benchpress #22390
Conversation
a5050b0
to
b543b49
Compare
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.
LGTM
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.
An issue with validating commits was found for when multiple level scopes as used and will be fixed with: angular/angular#41430 (review)
The dev-infra-private-builds version needs to include this change.
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.
LGTM
b543b49
to
44f82b0
Compare
@crisbeto needs rebase |
* Updates the `@angular` dependencies to the latest versions. * Removes the depenency on `@angular/benchpress` since we weren't using it and it was causing some issues in the snapshot builds.
44f82b0
to
0bd5cb9
Compare
Rebased. |
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. |
@angular
dependencies to the latest versions and resolves the build errors.@angular/benchpress
since we weren't using it and it was causing some issues in the snapshot builds.