Disable noUnusedLocals compiler option during script compilation

Author: paulbrimicombe

package.json

@@ -18,7 +18,7 @@
     "typescript",
     "verify"
   ],
-  "version": "1.0.1",
+  "version": "1.1.0",
   "main": "dist/index.js",
   "bin": "./dist/bin/compile-typescript-docs.js",
   "license": "Apache-2.0",

src/CodeBlockExtractor.ts

@@ -22,7 +22,7 @@ export class CodeBlockExtractor {
 
   private static extractCodeBlocksFromMarkdown (markdown: string): string[] {
     const codeBlocks: string[] = []
-    markdown.replace(this.TYPESCRIPT_CODE_PATTERN, (fullMatch, code) => {
+    markdown.replace(this.TYPESCRIPT_CODE_PATTERN, (_, code) => {
       codeBlocks.push(code)
       return code
     })

src/SnippetCompiler.ts

@@ -25,10 +25,20 @@ export class SnippetCompiler {
   private readonly runner: TypeScriptRunner
 
   constructor (private readonly workingDirectory: string) {
-    const configOptions = tsconfig.loadSync(process.cwd())
+    const configOptions = SnippetCompiler.loadTypeScriptConfig()
     this.runner = new TypeScriptRunner(this.workingDirectory, configOptions.config)
   }
 
+  private static loadTypeScriptConfig (): any {
+    const typeScriptConfig = tsconfig.loadSync(process.cwd())
+    if (typeScriptConfig &&
+        typeScriptConfig.config &&
+        typeScriptConfig.config.compilerOptions) {
+      typeScriptConfig.config.compilerOptions.noUnusedLocals = false
+    }
+    return typeScriptConfig
+  }
+
   compileSnippets (documentationFiles: string[]): Bluebird<SnippetCompilationResult[]> {
     return Bluebird.resolve()
       .then(() => this.cleanWorkingDirectory())

tsconfig.json

@@ -28,8 +28,8 @@
     "alwaysStrict": true,                     /* Parse in strict mode and emit "use strict" for each source file. */
 
     /* Additional Checks */                   
-    // "noUnusedLocals": true,                /* Report errors on unused locals. */
-    // "noUnusedParameters": true,            /* Report errors on unused parameters. */
+    "noUnusedLocals": true   ,                /* Report errors on unused locals. */
+    "noUnusedParameters": true,               /* Report errors on unused parameters. */
     "noImplicitReturns": true,                /* Report error when not all code paths in function return a value. */
     // "noFallthroughCasesInSwitch": true,    /* Report errors for fallthrough cases in switch statement. */