Skip to content
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

Remove "experimental" from zoneless and add prompt for new apps #30034

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

atscott
Copy link
Contributor

@atscott atscott commented Apr 4, 2025

See individual commits.

Blocked by angular/angular#60748

atscott added 2 commits April 4, 2025 13:26
This commit removes experimental from the option and provider names
This commit adds a prompt to ask if the user wants to generate the app
as zoneless.
@alfaproject
Copy link

OOC is there a world where we can have both zones and zoneless in the same project? Asking for the sake of incremental migrations

@atscott
Copy link
Contributor Author

atscott commented Apr 5, 2025

OOC is there a world where we can have both zones and zoneless in the same project? Asking for the sake of incremental migrations

No, not for that definition of “incremental”. But you can still incrementally migrate components to be Zoneless compatible. Everything Zoneless also works with zones. Everything in the angular components repo was migrated incrementally to be Zoneless compatible prior to the experimental release in v18.

@alfaproject
Copy link

Alright, so having only OnPush components, and signals, basically, right? I suppose you'll publish some tips/guides

@atscott
Copy link
Contributor Author

atscott commented Apr 6, 2025

https://angular.dev/guide/experimental/zoneless#requirements-for-zoneless-compatibility

OnPush, no NgZone.on[Stable/MicrotaskEmpty]. Test with the exhaustive checkNoChanges. It’s surprisingly straightforward. There should be minimal toil for apps written with OnPush. For those that aren’t, that step alone is 90% of the work and 99% of the difficulty.

@alfaproject
Copy link

Yup, awesome, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants