fix(generators/demo): avoid re-installing dependency

marviq · Oct 11, 2017 · c862fba · c862fba
1 parent 312f2de
commit c862fba
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/generators/app/index.js b/generators/app/index.js
@@ -702,7 +702,7 @@ class AppGenerator extends Generator
             '\n'
         +   'Running a number of "'
         +   chalk.yellow( 'npm install ' + chalk.cyan( '<package>' ) + ' --save' + chalk.cyan( '[' ) + '-dev' + chalk.cyan( ']' ))
-        +   '"s for you to install required dependencies.'
+        +   '"s for you to install required dependencies...\n'
         ));
 
         this.npmInstall( devDeps,   { saveDev:  true } );

diff --git a/generators/demo/index.js b/generators/demo/index.js
@@ -202,9 +202,22 @@ class DemoGenerator extends Generator
 
             if ( !( data.i18n )) { return; }
 
-            var deps        = [ 'madlib-locale' ];
+            var pkgPath     = this.destinationPath( 'package.json' )
+            ,   fs          = this.fs
+            ,   npm         = fs.readJSON( pkgPath )
+            ,   dep         = 'madlib-locale'
+            ;
+
+            if ( npm.dependencies[ dep ] ) { return; }
+
+            this.log( chalk.bold(
+                '\n'
+            +   'Running "'
+            +   chalk.yellow( 'npm install ' + chalk.cyan( dep ) + ' --save' )
+            +   '" for you to install required dependencies...\n'
+            ));
 
-            this.npmInstall( deps,      { save:     true } );
+            this.npmInstall( [ dep ],      { save:     true } );
         }
         )();