fix(hooks): don't require Gulpfile twice

tlancina · tlancina · commit c17d268f86b9 · 2016-03-28T08:35:24.000-05:00
diff --git a/lib/cli.js b/lib/cli.js
@@ -194,11 +194,10 @@ Cli.run = function run(processArgv) {
 
 function loadGulp(){
   var names = ['gulpfile.js', 'Gulpfile.js'];
-  var found = false;
   for (var i = 0, ii = names.length; i < ii; i++) {
     try {
       require(path.resolve(process.cwd() + '/' + names[i]));
-      found = true;
+      return true;
     } catch(e){
       if (e instanceof SyntaxError) {
         console.error('\nThere is a syntax error in your gulpfile:');
@@ -213,7 +212,7 @@ function loadGulp(){
       }
     }
   }
-  return found;
+  return false;
 }
 
 function logEvents(gulpInst) {

Original file line number	Diff line number	Diff line change
`@@ -194,11 +194,10 @@ Cli.run = function run(processArgv) {`
`194`	`194`
`195`	`195`	`function loadGulp(){`
`196`	`196`	`var names = ['gulpfile.js', 'Gulpfile.js'];`
`197`		`- var found = false;`
`198`	`197`	`for (var i = 0, ii = names.length; i < ii; i++) {`
`199`	`198`	`try {`
`200`	`199`	`require(path.resolve(process.cwd() + '/' + names[i]));`
`201`		`- found = true;`
	`200`	`+ return true;`
`202`	`201`	`} catch(e){`
`203`	`202`	`if (e instanceof SyntaxError) {`
`204`	`203`	`console.error('\nThere is a syntax error in your gulpfile:');`
`@@ -213,7 +212,7 @@ function loadGulp(){`
`213`	`212`	`}`
`214`	`213`	`}`
`215`	`214`	`}`
`216`		`- return found;`
	`215`	`+ return false;`
`217`	`216`	`}`
`218`	`217`
`219`	`218`	`function logEvents(gulpInst) {`