Skip to content

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Jun 10, 2024

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
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

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: 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
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.

Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

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
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
Copy link
Contributor

dylhunn commented Jun 25, 2024

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

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