Merge pull request #7042 from Microsoft/issue6941

Fixed watcher for rename operations, and for allowJs in config file
microsoft · Feb 12, 2016 · e7899cf · e7899cf
2 parents d5538f0 + 745f32b
commit e7899cf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/compiler/sys.ts b/src/compiler/sys.ts
@@ -381,7 +381,8 @@ namespace ts {
                     const filePath = typeof relativeFileName !== "string"
                         ? undefined
                         : toPath(relativeFileName, baseDirPath, createGetCanonicalFileName(sys.useCaseSensitiveFileNames));
-                    if (eventName === "change" && fileWatcherCallbacks.contains(filePath)) {
+                    // Some applications save a working file via rename operations
+                    if ((eventName === "change" || eventName === "rename") && fileWatcherCallbacks.contains(filePath)) {
                         for (const fileCallback of fileWatcherCallbacks.get(filePath)) {
                             fileCallback(filePath);
                         }

diff --git a/src/compiler/tsc.ts b/src/compiler/tsc.ts
@@ -486,7 +486,7 @@ namespace ts {
         }
 
         function watchedDirectoryChanged(fileName: string) {
-            if (fileName && !ts.isSupportedSourceFileName(fileName, commandLine.options)) {
+            if (fileName && !ts.isSupportedSourceFileName(fileName, compilerOptions)) {
                 return;
             }
-Original file line number
+Diff line change
@@ Expand Up / @@ -486,7 +486,7 @@ namespace ts { @@
             }
             function watchedDirectoryChanged(fileName: string) {
-                if (fileName && !ts.isSupportedSourceFileName(fileName, commandLine.options)) {
+                if (fileName && !ts.isSupportedSourceFileName(fileName, compilerOptions)) {
                     return;
                 }
@@ Expand Down @@