Skip to content

Comments

refactor(core): remove outdated TODO comments#67208

Merged
thePunderWoman merged 1 commit intoangular:mainfrom
MeAkib:fix-67191
Feb 23, 2026
Merged

refactor(core): remove outdated TODO comments#67208
thePunderWoman merged 1 commit intoangular:mainfrom
MeAkib:fix-67191

Conversation

@MeAkib
Copy link
Contributor

@MeAkib MeAkib commented Feb 23, 2026

Cleans up TODO comments that are no longer planned

Fixes #67191

Cleans up TODO comments that are no longer planned

Fixes angular#67191
@angular-robot angular-robot bot added area: core Issues related to the framework runtime requires: TGP This PR requires a passing TGP before merging is allowed labels Feb 23, 2026
@ngbot ngbot bot added this to the Backlog milestone Feb 23, 2026
@AndrewKushnir AndrewKushnir requested review from pkozlowski-opensource and removed request for AndrewKushnir February 23, 2026 17:17
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Feb 23, 2026
@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 Feb 23, 2026
@JeanMeche
Copy link
Member

Caretaker note: No need to TGP/presubmit this is a comment only change.

Copy link
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

reviewed-for: fw-general, primitives

@thePunderWoman
Copy link
Contributor

TESTED=Comment only

@thePunderWoman thePunderWoman merged commit 5ac1c02 into angular:main Feb 23, 2026
27 of 29 checks passed
@thePunderWoman
Copy link
Contributor

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

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

Labels

action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note PullApprove: disable requires: TGP This PR requires a passing TGP before merging is allowed target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor: Clean up signal file(s) TODO

3 participants