Skip to content

refactor: prepare for required changeDetection prop on G3.#67775

Merged
leonsenft merged 1 commit into
angular:mainfrom
JeanMeche:required-cd
Mar 20, 2026
Merged

refactor: prepare for required changeDetection prop on G3.#67775
leonsenft merged 1 commit into
angular:mainfrom
JeanMeche:required-cd

Conversation

@JeanMeche

@JeanMeche JeanMeche commented Mar 20, 2026

Copy link
Copy Markdown
Member

We'll make this a G3 only change to prevent backsliding during the transition period.

@JeanMeche
JeanMeche requested a review from atscott March 20, 2026 16:48
@JeanMeche

JeanMeche commented Mar 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Caretaker note: To sync with CL/884661058

@JeanMeche JeanMeche added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Mar 20, 2026
atscott
atscott reviewed Mar 20, 2026
View reviewed changes
Comment thread packages/core/src/metadata/directives.ts
@JeanMeche
refactor: prepare for required changeDetection prop on G3.
c3e25e7 
We'll make this a G3 only change to prevent backsliding during the transition period.
@leonsenft leonsenft added the area: core Issues related to the framework runtime label Mar 20, 2026
@ngbot ngbot Bot added this to the Backlog milestone Mar 20, 2026
@leonsenft leonsenft added area: router action: merge The PR is ready for merge by the caretaker action: global presubmit The PR is in need of a google3 global presubmit target: patch This PR is targeted for the next patch release and removed action: merge The PR is ready for merge by the caretaker action: global presubmit The PR is in need of a google3 global presubmit labels Mar 20, 2026
@leonsenft
leonsenft merged commit d2054c7 into angular:main Mar 20, 2026
29 of 31 checks passed
@leonsenft

leonsenft commented Mar 20, 2026

Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@JeanMeche
JeanMeche deleted the required-cd branch March 23, 2026 01:08
@angular-automatic-lock-bot

angular-automatic-lock-bot Bot commented Apr 23, 2026

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Apr 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@atscott atscott atscott approved these changes

Assignees

No one assigned

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime area: router merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release

Projects

None yet

Milestone

Backlog

Development

Successfully merging this pull request may close these issues.

3 participants

@JeanMeche @leonsenft @atscott