Skip to content

build: update to Angular 20 #4778

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

Merged
merged 13 commits into from
Jun 6, 2025
Merged

build: update to Angular 20 #4778

merged 13 commits into from
Jun 6, 2025

Conversation

timdeschryver
Copy link
Member

@timdeschryver timdeschryver commented May 12, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Closes #

What is the new behavior?

Does this PR introduce a breaking change?

[x] Yes
[ ] No

BREAKING CHANGES:

The minimum required version of Angular has been updated.

BEFORE:

The minimum required version is Angular 19.x

AFTER:

The minimum required version is Angular 20.x

Other information

While running pnpm i I noticed some old packages that can be updated/removed.
This can be done after this PR is merged.

Copy link

netlify bot commented May 12, 2025

Deploy Preview for ngrx-io canceled.

Name Link
🔨 Latest commit bf908a5
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-io/deploys/68429c7b0bbaa6000863ffdc

Copy link

netlify bot commented May 12, 2025

Deploy Preview for ngrx-site-v19 ready!

Name Link
🔨 Latest commit bf908a5
🔍 Latest deploy log https://app.netlify.com/projects/ngrx-site-v19/deploys/68429c7b7ec4a50008e7624b
😎 Deploy Preview https://deploy-preview-4778--ngrx-site-v19.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@markostanimirovic markostanimirovic added the Needs Cleanup Review changes needed label May 28, 2025
timdeschryver and others added 2 commits May 29, 2025 20:18
Co-authored-by: Marko Stanimirović <markostanimirovic95@gmail.com>
@timdeschryver timdeschryver changed the title build: update to Angular 20-rc build: update to Angular 20 May 29, 2025
@timdeschryver
Copy link
Member Author

I updated this PR to upgrade to Angular 20.
This causes some build problems that I will take care of tomorrow.

@timdeschryver timdeschryver force-pushed the ng20 branch 2 times, most recently from 96ec375 to dd89bbe Compare May 30, 2025 18:06
Copy link
Contributor

@rainerhahnekamp rainerhahnekamp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've checked out the branch. Ran some basic tests, new TUI looks great...all good!

@Julien-Marcou
Copy link

Is it possible to have a beta release with this PR so that we can test it on our side?

@timdeschryver
Copy link
Member Author

Is it possible to have a beta release with this PR so that we can test it on our side?

Yes, as usual there will be a beta version before the official release.

@JeevanMahesha
Copy link
Contributor

@timdeschryver when can we expect beta version and the official release date.

@timdeschryver
Copy link
Member Author

@JeevanMahesha see #4787

Copy link
Member Author

@timdeschryver timdeschryver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This cannot be merged yet, the docs publish needs to be fixed first.

@timdeschryver timdeschryver merged commit 8a4ecd9 into main Jun 6, 2025
14 checks passed
@timdeschryver timdeschryver deleted the ng20 branch June 6, 2025 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants