Skip to content

Conversation

clydin
Copy link
Member

@clydin clydin commented Oct 25, 2022

The experimental esbuild-based browser application builder will now avoid trying to query the Angular Compiler for template diagnostics when a TypeScript source file is a declaration file (.d.ts). This avoids the overhead of the in-memory diagnostics caching logic as well as any Angular Compiler logic to determine if the file has any template diagnostics.

…eclaration files in esbuild builder

The experimental esbuild-based browser application builder will now avoid trying to query the Angular
Compiler for template diagnostics when a TypeScript source file is a declaration file (`.d.ts`). This
avoids the overhead of the in-memory diagnostics caching logic as well as any Angular Compiler logic
to determine if the file has any template diagnostics.
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Oct 25, 2022
@clydin clydin requested a review from alan-agius4 October 25, 2022 14:42
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Oct 25, 2022
@dgp1130 dgp1130 merged commit 60086db into angular:main Oct 25, 2022
@clydin clydin deleted the esbuild/diag-perf branch October 25, 2022 17:53
@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 Nov 25, 2022
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: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants