feat: add support for Angular v22 release#2362
Conversation
✅ Deploy Preview for analog-app ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for analog-blog ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for analog-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
📝 WalkthroughWalkthroughThis pull request upgrades Angular dependencies across the Analog monorepo from release candidates (rc.0, rc.2) to stable 22.0.0. The root package.json updates framework runtime and development dependencies; core libraries (astro-angular, content, router) update peer dependency version ranges to permit stable versions; build tooling (vite-plugin-angular, vitest-angular) aligns with the new baseline; and all three project templates (blog, latest, minimal) match the root configuration. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This PR touches multiple package scopes: Please confirm the changes are closely related. |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
packages/vitest-angular/package.json (1)
37-37: 💤 Low valueRedundant overlapping range in
@angular-devkit/architect.
>=0.1700.0 < 0.2300.0already fully subsumes>=0.2200.0 < 0.2300.0, so the second clause is dead weight — almost certainly a leftover from dropping the-rc.0qualifier. No resolution impact, but it muddies the peer contract for maintainers.♻️ Collapse to the single effective range
- "`@angular-devkit/architect`": ">=0.1700.0 < 0.2300.0 || >=0.2200.0 < 0.2300.0", + "`@angular-devkit/architect`": ">=0.1700.0 < 0.2300.0",🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@packages/vitest-angular/package.json` at line 37, The peerDependency range for "`@angular-devkit/architect`" contains an overlapping redundant clause; replace the two-clause expression ("\">=0.1700.0 < 0.2300.0 || >=0.2200.0 < 0.2300.0\") with the single effective range ("\">=0.1700.0 < 0.2300.0\") in packages/vitest-angular's package.json so the peer contract is clear and no longer contains the dead-weight second clause.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In `@packages/vitest-angular/package.json`:
- Line 37: The peerDependency range for "`@angular-devkit/architect`" contains an
overlapping redundant clause; replace the two-clause expression ("\">=0.1700.0 <
0.2300.0 || >=0.2200.0 < 0.2300.0\") with the single effective range
("\">=0.1700.0 < 0.2300.0\") in packages/vitest-angular's package.json so the
peer contract is clear and no longer contains the dead-weight second clause.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: e384cb39-1174-4a42-88d3-ae9e0eecc9db
⛔ Files ignored due to path filters (1)
pnpm-lock.yamlis excluded by!**/pnpm-lock.yamland included by none
📒 Files selected for processing (9)
package.jsonpackages/astro-angular/package.jsonpackages/content/package.jsonpackages/create-analog/template-blog/package.jsonpackages/create-analog/template-latest/package.jsonpackages/create-analog/template-minimal/package.jsonpackages/router/package.jsonpackages/vite-plugin-angular/package.jsonpackages/vitest-angular/package.json
PR Checklist
Closes #
Affected scope
Recommended merge strategy for maintainer [optional]
Commit preservation note [optional]
What is the new behavior?
Angular v22 is supported across released packages.
Test plan
nx format:checkpnpm buildpnpm testDoes this PR introduce a breaking change?
Other information
[optional] What gif best describes this PR or how it makes you feel?