Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(language-service): avoid generating TS syntactic diagnostics for templates #55091

Closed
wants to merge 2 commits into from

Conversation

clydin
Copy link
Member

@clydin clydin commented Mar 28, 2024

Angular's template files are not valid TypeScript. Attempting to get syntactic diagnostics from the underlying TypeScript language service will result in a large amount of false positive errors. Only actual TypeScript files should be analyzed by the underlying TypeScript language service for syntactic errors.

…templates

Angular's template files are not valid TypeScript. Attempting to get syntactic
diagnostics from the underlying TypeScript language service will result in
a large amount of false positive errors. Only actual TypeScript files should
be analyzed by the underlying TypeScript language service for syntactic errors.
@clydin clydin added target: major This PR is targeted for the next major release action: review The PR is still awaiting reviews from at least one requested reviewer area: language-service Issues related to Angular's VS Code language service labels Mar 28, 2024
@ngbot ngbot bot modified the milestone: Backlog Mar 28, 2024
@clydin clydin marked this pull request as ready for review March 28, 2024 17:47
@pullapprove pullapprove bot requested a review from atscott March 28, 2024 17:47
@clydin clydin added the merge: squash commits When the PR is merged, a squash and merge should be performed label Mar 28, 2024
@atscott atscott 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 Mar 28, 2024
@atscott
Copy link
Contributor

atscott commented Mar 28, 2024

This PR was merged into the repository by commit a48afe0.

@atscott atscott closed this in a48afe0 Mar 28, 2024
@clydin clydin deleted the ls/template-syntax-diags branch March 28, 2024 19:11
ilirbeqirii pushed a commit to ilirbeqirii/angular that referenced this pull request Apr 6, 2024
…templates (angular#55091)

Angular's template files are not valid TypeScript. Attempting to get syntactic
diagnostics from the underlying TypeScript language service will result in
a large amount of false positive errors. Only actual TypeScript files should
be analyzed by the underlying TypeScript language service for syntactic errors.

PR Close angular#55091
@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 Apr 28, 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: language-service Issues related to Angular's VS Code language service merge: squash commits When the PR is merged, a squash and merge should be performed target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants