Merge pull request #1576 from paulvanbrenk/fixTypingsAqq

Make sure we send options for the install_typings in both cases.
microsoft · May 12, 2017 · e06b3a8 · e06b3a8
2 parents a04a535 + 5dce673
commit e06b3a8
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/Nodejs/Product/Nodejs/TypingsAcquisitionTool/bin/install_typings b/Nodejs/Product/Nodejs/TypingsAcquisitionTool/bin/install_typings
@@ -17,12 +17,14 @@ var argv = minimist(process.argv.slice(2), {
 });
 
 var emitter = new events.EventEmitter();
-
 var packagesToInstall = argv._;
 
 if (!packagesToInstall.length) {
     // top level package install
-    typingsTool.installTypingsForProject(options)
+    typingsTool.installTypingsForProject({ save: argv.save,
+                                           emitter: emitter,
+                                           global: true,
+                                           cwd: argv.cwd || process.cwd()})
 } else {
     typingsTool.runAll(packagesToInstall.map(function (name) {
         var options = {
@@ -31,6 +33,7 @@ if (!packagesToInstall.length) {
             global: name === "node", // Assume everything else refers to a CommonJS module
             cwd: argv.cwd || process.cwd()
         };
+
         return typingsTool.installTypingsForPackage(name, options);
     }));
 }