-
Notifications
You must be signed in to change notification settings - Fork 315
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
chore(angular): update dependencies for v19, use latest control flow syntax and build #1858
base: main
Are you sure you want to change the base?
Conversation
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.
Typescript needs to be pinned to minor here because version 5.7.2
causes errors with vue-tsc. This prevents the build from passing on this PR: https://github.com/ionic-team/starters/actions/runs/12321151698/job/34391988149?pr=1858
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.
Since this file is already being edited, can we add a new line to the end?
angular
andangular-standalone
to Angular v19@ionic/angular-toolkit
to v12eslint
to v9@for
instead of*ngFor
)angular-standalone
to the new build system via theapplication
builder