Skip to content

feat(core): drop support for TypeScript older than 5.8#60197

Closed
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:drop-ts-5.7
Closed

feat(core): drop support for TypeScript older than 5.8#60197
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:drop-ts-5.7

Conversation

@crisbeto
Copy link
Copy Markdown
Member

@crisbeto crisbeto commented Mar 4, 2025

Narrows down the versions of TypeScript we need to support.

BREAKING CHANGE:

  • TypeScript versions less than 5.8 are no longer supported.

@crisbeto crisbeto added action: review The PR is still awaiting reviews from at least one requested reviewer target: major This PR is targeted for the next major release labels Mar 4, 2025
@angular-robot angular-robot bot added detected: breaking change PR contains a commit with a breaking change detected: feature PR contains a feature commit area: core Issues related to the framework runtime labels Mar 4, 2025
@ngbot ngbot bot added this to the Backlog milestone Mar 4, 2025
@JoostK
Copy link
Copy Markdown
Member

JoostK commented Mar 4, 2025

The PR title/commit message scared me for a bit. Versions older than TypeScript 5.8 😄

@crisbeto crisbeto changed the title feat(core): drop support for TypeScript 5.8 feat(core): drop support for TypeScript older than 5.8 Mar 4, 2025
@crisbeto crisbeto marked this pull request as ready for review March 4, 2025 08:14
@pullapprove pullapprove bot requested a review from devversion March 4, 2025 08:14
@crisbeto
feat(core): drop support for TypeScript older than 5.8
5525895 
Narrows down the versions of TypeScript we need to support.

BREAKING CHANGE:
* TypeScript versions less than 5.8 are no longer supported.
@crisbeto crisbeto added merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 4, 2025
@crisbeto
Copy link
Copy Markdown
Member Author

crisbeto commented Mar 4, 2025

Caretaker note: the lint job is failing because of an issue with the license checks. Dev infra has said that we can safely ignore it.

@mmalerba
Copy link
Copy Markdown
Contributor

mmalerba commented Mar 4, 2025

This PR was merged into the repository by commit 326d48a.

The changes were merged into the following branches: main

@mmalerba mmalerba closed this in 326d48a Mar 4, 2025
@angular-automatic-lock-bot
Copy link
Copy Markdown

angular-automatic-lock-bot bot commented Apr 4, 2025

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 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@devversion devversion devversion 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 detected: breaking change PR contains a commit with a breaking change detected: feature PR contains a feature commit merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: major This PR is targeted for the next major release

Projects

None yet

Milestone

Backlog

Development

Successfully merging this pull request may close these issues.

4 participants

@crisbeto @JoostK @mmalerba @devversion