Skip to content

build: update to TypeScript 5.5 stable - #56358

Closed
crisbeto wants to merge 1 commit into
angular:mainfrom
crisbeto:ts-5.5-rc
Closed

crisbeto wants to merge 1 commit into
angular:mainfrom
crisbeto:ts-5.5-rc

Conversation

@crisbeto

@crisbeto crisbeto commented Jun 10, 2024 •

Copy link
Copy Markdown
Member

Updates the repo to the stable release of TypeScript 5.5.

@crisbeto crisbeto added area: build & ci Related the build and CI infrastructure of the project target: minor This PR is targeted for the next minor release labels Jun 10, 2024
@ngbot ngbot Bot added this to the Backlog milestone Jun 10, 2024
@crisbeto
crisbeto force-pushed the ts-5.5-rc branch 2 times, most recently from 8224da2 to fe7459e Compare June 10, 2024 21:41
@crisbeto crisbeto added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jun 10, 2024
@crisbeto
crisbeto marked this pull request as ready for review June 10, 2024 21:41
@pullapprove
pullapprove Bot requested a review from josephperrott June 10, 2024 21:41

@josephperrott josephperrott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thePunderWoman thePunderWoman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

reviewed-for: public-api

@pullapprove
pullapprove Bot requested a review from AndrewKushnir June 11, 2024 18:39
@pullapprove
pullapprove Bot requested a review from AndrewKushnir June 11, 2024 23:14
@crisbeto

Copy link
Copy Markdown
Member Author

I'm holding this PR back for a bit because we're still talking with Alex if it's the right way to resolve one of the issues that popped up.

@pkozlowski-opensource pkozlowski-opensource left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: public-api

crisbeto added a commit to crisbeto/angular-cli that referenced this pull request Jun 21, 2024
Updates the repo to the stable version of TypeScript 5.5. I'm holding off on updating the new projects to it due to angular/angular#56358.
crisbeto added a commit to crisbeto/angular-cli that referenced this pull request Jun 21, 2024
Updates the repo to the stable version of TypeScript 5.5. I'm holding off on updating the new projects to it due to angular/angular#56358.
@crisbeto crisbeto changed the title build: update to the TypeScript 5.5 RC build: update to the TypeScript 5.5 stable Jun 21, 2024
@crisbeto crisbeto changed the title build: update to the TypeScript 5.5 stable build: update to TypeScript 5.5 stable Jun 21, 2024
crisbeto added a commit to crisbeto/angular-cli that referenced this pull request Jun 21, 2024
Updates the repo to the stable version of TypeScript 5.5. I'm holding off on updating the new projects to it due to angular/angular#56358.
alan-agius4 pushed a commit to angular/angular-cli that referenced this pull request Jun 21, 2024
Updates the repo to the stable version of TypeScript 5.5. I'm holding off on updating the new projects to it due to angular/angular#56358.
@crisbeto
crisbeto force-pushed the ts-5.5-rc branch 2 times, most recently from 42a0022 to 1afa6fe Compare June 24, 2024 08:17
Updates the repo to the stable release of TypeScript 5.5.
getSourceFile(fileName: string, languageVersion: ts.ScriptTarget): ts.SourceFile | undefined {
const text = this.readFile(fileName);
return text !== undefined
? ts.createSourceFile(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The context here is that I had to make this change during the update to the TS 5.5 beta, but it's no longer necessary for the RC/final.

@crisbeto
crisbeto removed the request for review from AndrewKushnir June 24, 2024 18:45
@crisbeto crisbeto added 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 Jun 24, 2024
@dylhunn

dylhunn commented Jun 25, 2024

Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit ccc8c80.

The changes were merged into the following branches: main

@dylhunn dylhunn closed this in ccc8c80 Jun 25, 2024
crisbeto added a commit to crisbeto/angular-cli that referenced this pull request Jun 26, 2024
Now that angular/angular#56358 is on track for 18.1, we can use TypeScript 5.5 for newly-created projects and in tests.
crisbeto added a commit to crisbeto/angular-cli that referenced this pull request Jun 26, 2024
Now that angular/angular#56358 is on track for 18.1, we can use TypeScript 5.5 for newly-created projects and in tests.
dgp1130 pushed a commit to angular/angular-cli that referenced this pull request Jun 26, 2024
Now that angular/angular#56358 is on track for 18.1, we can use TypeScript 5.5 for newly-created projects and in tests.
crisbeto added a commit to crisbeto/angular that referenced this pull request Jul 12, 2024
… 5.4

In angular#56358 we removed most of the places that untag the references to typecheck files, because it was causing the compiler to throw error when it produces diagnostics. This appears to have caused a regression in TS 5.4 which now emits the synthetic references.

These changes add tagging right before the program emits.

Fixes angular#56945.
atscott pushed a commit that referenced this pull request Jul 15, 2024
… 5.4 (#56961)

In #56358 we removed most of the places that untag the references to typecheck files, because it was causing the compiler to throw error when it produces diagnostics. This appears to have caused a regression in TS 5.4 which now emits the synthetic references.

These changes add tagging right before the program emits.

Fixes #56945.

PR Close #56961
atscott pushed a commit that referenced this pull request Jul 15, 2024
… 5.4 (#56961)

In #56358 we removed most of the places that untag the references to typecheck files, because it was causing the compiler to throw error when it produces diagnostics. This appears to have caused a regression in TS 5.4 which now emits the synthetic references.

These changes add tagging right before the program emits.

Fixes #56945.

PR Close #56961
@angular-automatic-lock-bot

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

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants