Skip to content

Eager Change Detection#32737

Open
alan-agius4 wants to merge 2 commits intoangular:mainfrom
alan-agius4:eager-change-detection
Open

Eager Change Detection#32737
alan-agius4 wants to merge 2 commits intoangular:mainfrom
alan-agius4:eager-change-detection

Conversation

@alan-agius4
Copy link
Collaborator

No description provided.

@alan-agius4 alan-agius4 requested a review from clydin March 10, 2026 14:46
@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Mar 10, 2026
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 10, 2026
….Default` with `Eager`

Updates the component schematic to use `ChangeDetectionStrategy.Eager` as
the default strategy and updates the allowed enum values in the schema,
as `ChangeDetectionStrategy.Default` is now deprecated.
@alan-agius4 alan-agius4 force-pushed the eager-change-detection branch from 9e37d7c to e2508f6 Compare March 10, 2026 15:16
…tectionStrategy.Eager`

`ChangeDetectionStrategy.Default` is deprecated. This update ensures the zoneless migration tool correctly identifies, handles, and prompts for the new `ChangeDetectionStrategy.Eager` strategy in component metadata.
@alan-agius4 alan-agius4 force-pushed the eager-change-detection branch from e2508f6 to 9ea5868 Compare March 10, 2026 15:46
},
"changeDetection": {
"description": "Configures the change detection strategy for the component.",
"enum": ["Default", "OnPush"],
Copy link
Member

Choose a reason for hiding this comment

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

Should we leave Default here until its removed on the framework side?

Copy link
Collaborator Author

@alan-agius4 alan-agius4 Mar 10, 2026

Choose a reason for hiding this comment

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

I removed it, cause the FW already did a migration to move all apps from using Default to Eager.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: review The PR is still awaiting reviews from at least one requested reviewer area: @angular/cli area: @schematics/angular target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants