From 061eda5341967a24692ab587b91d35ff54fbbb89 Mon Sep 17 00:00:00 2001
From: Anton Gilgur <agilgur5@gmail.com>
Date: Sat, 11 Apr 2020 21:50:04 -0400
Subject: [PATCH] (fix): strict* checks are true when `strict`

- in the docs, "Strict Checks" include noImplicitAny, noImplicitThis,
  strictNullChecks, strictFunctionTypes, strictBindCallApply,
  strictPropertyInitialization, and alwaysStrict
  - so they should all have consistent defaults

- noImplicitReturns is *not* part of "Strict Checks" and is actually
  part of "Linter Checks", so this was incorrectly documented
  - so its default should be plain false
---
 packages/tsconfig-reference/scripts/tsconfigRules.ts | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/packages/tsconfig-reference/scripts/tsconfigRules.ts b/packages/tsconfig-reference/scripts/tsconfigRules.ts
index b04e133c3c43..28d32a5c8337 100644
--- a/packages/tsconfig-reference/scripts/tsconfigRules.ts
+++ b/packages/tsconfig-reference/scripts/tsconfigRules.ts
@@ -126,7 +126,7 @@ export const defaultsForOptions = {
   noErrorTruncation: "false",
   noFallthroughCasesInSwitch: "false",
   noImplicitAny: "`false`, unless `strict` is set",
-  noImplicitReturns: "`false`, unless `strict` is set",
+  noImplicitReturns: "false",
   noImplicitThis: "`false`, unless `strict` is set",
   noImplicitUseStrict: "false",
   noLib: "false",
@@ -149,10 +149,10 @@ export const defaultsForOptions = {
   skipLibCheck: "false",
   sourceMap: "false",
   strict: "false",
-  strictBindCallApply: "false",
-  strictFunctionTypes: "false",
-  strictPropertyInitialization: "false",
-  strictNullChecks: "false",
+  strictBindCallApply: "`false`, unless `strict` is set",
+  strictFunctionTypes: "`false`, unless `strict` is set",
+  strictPropertyInitialization: "`false`, unless `strict` is set",
+  strictNullChecks: "`false`, unless `strict` is set",
   suppressExcessPropertyErrors: "false",
   suppressImplicitAnyIndexErrors: "false",
   target: "false",