Skip to content

Commit

Permalink
Enable strictNullChecks for implicit projects (#149844)
Browse files Browse the repository at this point in the history
Mistakenly enabled checkJS instead of strictNullChecks
  • Loading branch information
mjbvz committed May 18, 2022
1 parent ff975bc commit 6bd7605
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions extensions/typescript-language-features/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -686,7 +686,7 @@
},
"js/ts.implicitProjectConfig.checkJs": {
"type": "boolean",
"default": true,
"default": false,
"markdownDescription": "%configuration.implicitProjectConfig.checkJs%",
"scope": "window"
},
Expand All @@ -705,7 +705,7 @@
},
"js/ts.implicitProjectConfig.strictNullChecks": {
"type": "boolean",
"default": false,
"default": true,
"markdownDescription": "%configuration.implicitProjectConfig.strictNullChecks%",
"scope": "window"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ export class ImplicitProjectConfiguration {

private static readCheckJs(configuration: vscode.WorkspaceConfiguration): boolean {
return configuration.get<boolean>('js/ts.implicitProjectConfig.checkJs')
?? configuration.get<boolean>('javascript.implicitProjectConfig.checkJs', true);
?? configuration.get<boolean>('javascript.implicitProjectConfig.checkJs', false);
}

private static readExperimentalDecorators(configuration: vscode.WorkspaceConfiguration): boolean {
Expand All @@ -91,7 +91,7 @@ export class ImplicitProjectConfiguration {
}

private static readImplicitStrictNullChecks(configuration: vscode.WorkspaceConfiguration): boolean {
return configuration.get<boolean>('js/ts.implicitProjectConfig.strictNullChecks', false);
return configuration.get<boolean>('js/ts.implicitProjectConfig.strictNullChecks', true);
}

private static readImplicitStrictFunctionTypes(configuration: vscode.WorkspaceConfiguration): boolean {
Expand Down

0 comments on commit 6bd7605

Please sign in to comment.