Merge pull request #572 from stepancar/feature

refs #531 noEmit fix in atom-typescript. do not emit files if noEmit …

Author: basarat

dist/main/lang/modules/building.js

@@ -38,7 +38,7 @@ function emitFile(proj, filePath) {
         output.outputFiles.forEach(function (o) {
             mkdirp.sync(path.dirname(o.name));
             var additionalEmits = runExternalTranspiler(filePath, sourceFile.text, o, proj, sourceMapContents);
-            if (!sourceMapContents[o.name]) {
+            if (!sourceMapContents[o.name] && !proj.projectFile.project.compilerOptions.noEmit) {
                 fs.writeFileSync(o.name, o.text, "utf8");
             }
             additionalEmits.forEach(function (a) {

lib/main/lang/modules/building.ts

@@ -62,7 +62,7 @@ export function emitFile(proj: project.Project, filePath: string): EmitOutput {
                 sourceMapContents
             );
 
-            if (!sourceMapContents[o.name]) {
+            if (!sourceMapContents[o.name] && !proj.projectFile.project.compilerOptions.noEmit) {
                 // .js.map files will be written as an "additional emit" later.
                 fs.writeFileSync(o.name, o.text, "utf8");
             }
@@ -74,7 +74,7 @@ export function emitFile(proj: project.Project, filePath: string): EmitOutput {
         });
     }
 
-    // There is no *official* emit output for a `d.ts` 
+    // There is no *official* emit output for a `d.ts`
     // but its nice to have a consistent world view in the rest of our code
     var outputFiles = output.outputFiles.map((o) => o.name);
     if (path.extname(filePath) == '.d.ts') {
@@ -264,4 +264,4 @@ export function emitDts(proj: project.Project) {
 
         outDir: proj.projectFile.project.compilerOptions.outDir
     })
-}
+}