Copilot commented on code in PR #5861:
URL: https://github.com/apache/texera/pull/5861#discussion_r3449147449
##########
frontend/package.json:
##########
@@ -26,10 +26,10 @@
"@angular/animations": "21.2.10",
"@angular/cdk": "21.2.8",
"@angular/common": "21.2.17",
- "@angular/compiler": "21.2.10",
+ "@angular/compiler": "21.2.17",
"@angular/core": "21.2.17",
"@angular/forms": "21.2.10",
- "@angular/localize": "21.2.10",
+ "@angular/localize": "21.2.17",
Review Comment:
The Angular runtime packages are now split across patch versions (e.g.,
compiler/common/core/localize at 21.2.17, but
animations/forms/platform-browser/platform-browser-dynamic/router still at
21.2.10). Angular packages in this lockfile declare exact-version
peerDependencies (e.g. forms/platform-browser(-dynamic) require 21.2.10 peers),
so this bump is likely to introduce Yarn peer-dependency warnings and can lead
to hard-to-debug runtime/build issues.
Consider bumping the remaining @angular/* runtime deps to 21.2.17 as well
(and re-generating yarn.lock) so the Angular suite stays on one patch version.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
