build: update cross-repo angular dependencies#3576
Conversation
angular-robot
bot
added
the
area: build & ci
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates various Angular and related package versions across several package.json and pnpm-lock.yaml files, moving most @angular packages from 22.0.0-next.4 to 22.0.0-next.5 (or equivalent next patch/minor versions for other packages like @angular/cdk, @angular/build, @angular/cli, @angular/ssr, @angular-devkit/*, @schematics/angular, @listr2/prompt-adapter-inquirer, @oxc-project/types, and rolldown). Additionally, bazel/spec-bundling/test/async-await.spec.ts is updated to import ChangeDetectionStrategy and set changeDetection: ChangeDetectionStrategy.Eager for AppComponent. A critical issue was identified where the @octokit/rest dependency in apps/code-of-conduct/package.json has an invalid empty version string, which needs to be corrected.
| "@angular/animations": "22.0.0-next.5", | ||
| "@angular/cdk": "22.0.0-next.2", | ||
| "@angular/common": "22.0.0-next.5", | ||
| "@angular/compiler": "22.0.0-next.5", | ||
| "@angular/core": "22.0.0-next.5", | ||
| "@angular/fire": "21.0.0-rc.0", | ||
| "@angular/forms": "22.0.0-next.4", | ||
| "@angular/material": "22.0.0-next.1", | ||
| "@angular/platform-browser": "22.0.0-next.4", | ||
| "@angular/router": "22.0.0-next.4", | ||
| "@angular/forms": "22.0.0-next.5", | ||
| "@angular/material": "22.0.0-next.2", | ||
| "@angular/platform-browser": "22.0.0-next.5", | ||
| "@angular/router": "22.0.0-next.5", |
There was a problem hiding this comment.
The dependency @octokit/rest on line 13 has an empty version string (""), which is invalid for a package.json file. This will likely cause package installation to fail. Please specify a valid version for this package or remove it if it's no longer needed.
See associated pull request for more information. Closes angular#3572 as a pr takeover
alan-agius4
force-pushed
the
pr-takeover-3572
branch
from
deaa129 to
6132190
Compare
|
This PR was merged into the repository. The changes were merged into the following branches:
|
2 participants
See associated pull request for more information.
Closes #3572 as a pr takeover