Speculative fix for #568 (#569)

* Speculative fix for #568 * Remove default of setting isolatedModules to true when in transpileOnly mode Not useful for when using ts-loader with fork-ts-checker-webpack-plugin. Can still be set manually with tsconfig.json if desired and so arguably not a breaking change
TypeStrong · Jul 1, 2017 · 7004e17 · 7004e17
1 parent 767f796
commit 7004e17
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/src/config.ts b/src/config.ts
@@ -54,11 +54,6 @@ export function getConfigFile(
         configFile.config.compilerOptions = Object.assign({},
             configFile.config.compilerOptions,
             loaderOptions.compilerOptions);
-
-        // do any necessary config massaging
-        if (loaderOptions.transpileOnly) {
-            configFile.config.compilerOptions.isolatedModules = true;
-        }
     }
 
     return {

diff --git a/src/servicesHost.ts b/src/servicesHost.ts
@@ -65,8 +65,8 @@ function makeServicesHost(
         directoryExists: compiler.sys ? (<any> compiler.sys).directoryExists : undefined,
 
         // The following three methods are necessary for @types resolution from TS 2.4.1 onwards see: https://github.com/Microsoft/TypeScript/issues/16772
-        fileExists: moduleResolutionHost.fileExists,
-        readFile: moduleResolutionHost.readFile,
+        fileExists: compiler.sys ? (<any> compiler.sys).fileExists : undefined,
+        readFile: compiler.sys ? (<any> compiler.sys).readFile : undefined,
         readDirectory: compiler.sys ? (<any> compiler.sys).readDirectory : undefined,
 
         getCurrentDirectory: () => process.cwd(),