Enable all the ts checks we're now passing (#1044)

Author: EzraBrooks

tsconfig.build.json

@@ -33,22 +33,22 @@
     "alwaysStrict": true /* Ensure 'use strict' is always emitted. */,
     "noFallthroughCasesInSwitch": true /* Enable error reporting for fallthrough cases in switch statements */,
     "strictNullChecks": true /* When type checking, take into account `null` and `undefined`. */,
+    "noImplicitReturns": true /* Enable error reporting for codepaths that do not explicitly return in a function. */,
+    "noImplicitThis": true /* Enable error reporting when `this` is given the type `any`. */,
+    "noPropertyAccessFromIndexSignature": true /* Enforces using indexed accessors for keys declared using an indexed type */,
+    "noUnusedLocals": true /* Enable error reporting when a local variable isn't read. */,
+    "noUnusedParameters": true /* Raise an error when a function parameter isn't read */,
+    "strictBindCallApply": true /* Check that the arguments for `bind`, `call`, and `apply` methods match the original function. */,
+    "strictBuiltinIteratorReturn": true /* Built-in iterators are instantiated with a 'TReturn' type of 'undefined' instead of 'any'. */,
+    "useUnknownInCatchVariables": true /* Default catch clause variables as `unknown` instead of `any`. */,
 
     /* Rules to be enabled */
     "exactOptionalPropertyTypes": false /* Differentiate between undefined and not present when type checking */,
     "noImplicitAny": false /* Enable error reporting for expressions and declarations with an implied 'any' type. */,
     "noImplicitOverride": false /* Ensure overriding members in derived classes are marked with an override modifier. */,
-    "noImplicitReturns": false /* Enable error reporting for codepaths that do not explicitly return in a function. */,
-    "noImplicitThis": false /* Enable error reporting when `this` is given the type `any`. */,
-    "noPropertyAccessFromIndexSignature": false /* Enforces using indexed accessors for keys declared using an indexed type */,
     "noUncheckedIndexedAccess": false /* Add `undefined` to a type when accessed using an index. */,
-    "noUnusedLocals": false /* Enable error reporting when a local variable isn't read. */,
-    "noUnusedParameters": false /* Raise an error when a function parameter isn't read */,
-    "strictBindCallApply": false /* Check that the arguments for `bind`, `call`, and `apply` methods match the original function. */,
-    "strictBuiltinIteratorReturn": false /* Built-in iterators are instantiated with a 'TReturn' type of 'undefined' instead of 'any'. */,
     "strictFunctionTypes": false /* When assigning functions, check to ensure parameters and the return values are subtype-compatible. */,
     "strictPropertyInitialization": false /* Check for class properties that are declared but not set in the constructor. */,
-    "useUnknownInCatchVariables": false /* Default catch clause variables as `unknown` instead of `any`. */,
 
     "types": ["@types/node"]
   },