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(compiler-cli): Catch FatalDiagnosticError during template type ch… #49527
Conversation
c8882d1
to
bba17f1
Compare
caretaker note: CI has pre-existing failures at the moment. g3 tests passed. That seems like a good indication that this should be good to go. But feel free to just wait until CI is back in a good state. |
@atscott the CI should be in good state right now, could you please rebase this PR (to rerun the CI again)? It looks like this PR wouldn't merge cleanly into 15.2.x. We can merge this PR to 16.0.x and main and we'd need a separate PR for 15.2.x. |
…ecking This commit updates the type checking operation to catch `FatalDiagnosticError` and surface them as diagnostics rather than crashing. Fixes angular/vscode-ng-language-service#1881
…s not exported The compiler currently does not check to make sure that directives in the host bindings are exported. These directives are part of the public API of the component so they do have to be.
…nding is not exported
This PR was merged into the repository by commit e949548. |
…ecking (#49527) This commit updates the type checking operation to catch `FatalDiagnosticError` and surface them as diagnostics rather than crashing. Fixes angular/vscode-ng-language-service#1881 PR Close #49527
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…ecking
This commit updates the type checking step of the compilation to catch
FatalDiagnosticError
and surface them as diagnostics rather than crashing.Fixes angular/vscode-ng-language-service#1881