Enable Semantics Diagnostics for Inline JS (#25850)

Fixes #25809

**Bug**
Inline js in html currently only reports semantic errors. This means that `// @ts-check` does not work to help catch programming errors

**Fix**
Also report semantic errors in script tags
This commit is contained in:
Matt Bierner 2017-05-03 15:23:04 -07:00 committed by GitHub
parent 242cb936f0
commit 9b0141aaec

View file

@ -71,8 +71,9 @@ export function getJavascriptMode(documentRegions: LanguageModelCache<HTMLDocume
},
doValidation(document: TextDocument): Diagnostic[] {
updateCurrentTextDocument(document);
const diagnostics = jsLanguageService.getSyntacticDiagnostics(FILE_NAME);
return diagnostics.map((diag): Diagnostic => {
const syntaxDiagnostics = jsLanguageService.getSyntacticDiagnostics(FILE_NAME);
const semanticDiagnostics = jsLanguageService.getSemanticDiagnostics(FILE_NAME);
return syntaxDiagnostics.concat(semanticDiagnostics).map((diag): Diagnostic => {
return {
range: convertRange(currentTextDocument, diag),
severity: DiagnosticSeverity.Error,