Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

perf(@angular-devkit/build-angular): reduce TypeScript JSDoc parsing in application builder#26463

Merged
clydin merged 1 commit intoangular:mainangular/angular-cli:mainfrom
clydin:esbuild/jsdocparsemodeclydin/angular-cli:esbuild/jsdocparsemodeCopy head branch name to clipboard
Dec 8, 2023
Merged

perf(@angular-devkit/build-angular): reduce TypeScript JSDoc parsing in application builder#26463
clydin merged 1 commit intoangular:mainangular/angular-cli:mainfrom
clydin:esbuild/jsdocparsemodeclydin/angular-cli:esbuild/jsdocparsemodeCopy head branch name to clipboard

Conversation

@clydin
Copy link
Member

@clydin clydin commented Nov 23, 2023

TypeScript 5.3 provides a new programmatically accessible option on the compiler host object to control the amount of JSDoc parsing that the TypeScript parser will perform. The tsc command line tool now uses the ParseForTypeErrors value which only parses JSDoc comments that may affect type checking and is considered a good default for tools such as the Angular CLI. The Angular CLI will now attempt to use the ParseForTypeErrors value as well when available. Projects will need to use TypeScript 5.3+ for this option to be available. No behavior changes will occur on TypeScript 5.2 projects. This should not only provide a small improvement to build times but also a reduction in overall memory usage.

@clydin clydin added state: blocked target: minor This PR is targeted for the next minor release labels Nov 23, 2023
@clydin clydin force-pushed the esbuild/jsdocparsemode branch 2 times, most recently from f6358db to 980acd4 Compare November 30, 2023 18:19
…in application builder

TypeScript 5.3 provides a new programmatically accessible option on the compiler host object
to control the amount of JSDoc parsing that the TypeScript parser will perform. The `tsc`
command line tool now uses the `ParseForTypeErrors` value which only parses JSDoc comments
that may affect type checking and is considered a good default for tools such as the Angular
CLI. The Angular CLI will now attempt to use the `ParseForTypeErrors` value as well when available.
Projects will need to use TypeScript 5.3+ for this option to be available. No behavior changes will
occur on TypeScript 5.2 projects. This should not only provide a small improvement to build times
but also a reduction in overall memory usage.
@clydin clydin force-pushed the esbuild/jsdocparsemode branch from 980acd4 to 37ee033 Compare December 6, 2023 19:45
@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer and removed state: blocked labels Dec 6, 2023
@clydin clydin requested a review from alan-agius4 December 6, 2023 20:14
@alan-agius4 alan-agius4 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 Dec 8, 2023
@clydin clydin merged commit 125fb77 into angular:main Dec 8, 2023
@clydin clydin deleted the esbuild/jsdocparsemode branch December 8, 2023 18:03
@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 Jan 8, 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 target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.