Skip to content

refactor(core): remove outdated TODO comments referencing TypeScript 2.1#67069

Open
Rajveer173 wants to merge 1 commit intoangular:mainfrom
Rajveer173:refactor/remove-outdated-ts2.1-todos
Open

refactor(core): remove outdated TODO comments referencing TypeScript 2.1#67069
Rajveer173 wants to merge 1 commit intoangular:mainfrom
Rajveer173:refactor/remove-outdated-ts2.1-todos

Conversation

@Rajveer173
Copy link

@Rajveer173 Rajveer173 commented Feb 14, 2026

PR Checklist

PR Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other

What is the current behavior?

The files keyvalue_differs.ts and ng_for_of.ts contain outdated TODO comments referencing TypeScript 2.1 features. Angular now requires TypeScript >= 5.9, making those comments obsolete.

Issue Number: N/A

What is the new behavior?

Removed outdated TypeScript 2.1 TODO comments.
No functional changes.
No API changes.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

This change removes legacy comments that are no longer relevant and improves maintainability.

Angular requires TypeScript >=5.9, making TODO comments that reference TypeScript 2.1 features obsolete. These comments existed in keyvalue_differs.ts and ng_for_of.ts and referenced type improvements that were deferred until TS 2.1, which has long since been surpassed.
@pullapprove pullapprove bot requested a review from mmalerba February 14, 2026 12:29
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Feb 14, 2026
@ngbot ngbot bot added this to the Backlog milestone Feb 14, 2026
Copy link
Member

@JeanMeche JeanMeche left a comment

Choose a reason for hiding this comment

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

Yeah, it's unlikely we'll do that change.

@JeanMeche JeanMeche removed the request for review from mmalerba February 14, 2026 12:46
@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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels Feb 14, 2026
@JeanMeche
Copy link
Member

Caretaker note: This is comment only change, this doesn't need a presubmit.

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 target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants