Restore lint default

Author: mde

.eslintrc.js

@@ -24,6 +24,7 @@ module.exports = {
   },
   extends: 'eslint:recommended',
   rules: {
+    'semi': ["error", "always"],
     'indent': ['error', 2],
     'no-console': 'off',
     'no-empty': 'off',

lib/parseargs.js

@@ -17,8 +17,8 @@
 */
 
 let parseargs = {};
-let isOpt = function (arg) { return arg.indexOf('-') === 0 };
-let removeOptPrefix = function (opt) { return opt.replace(/^--/, '').replace(/^-/, '') };
+let isOpt = function (arg) { return arg.indexOf('-') === 0; };
+let removeOptPrefix = function (opt) { return opt.replace(/^--/, '').replace(/^-/, ''); };
 
 /**
  * @constructor

lib/task/task.js

@@ -122,7 +122,7 @@ class Task extends EventEmitter {
     this._initInvocationChain();
 
     this.args = Array.prototype.slice.call(arguments);
-    this.reenabled = false
+    this.reenabled = false;
     this.runPrereqs();
   }
 
@@ -137,7 +137,7 @@ class Task extends EventEmitter {
 
     this.args = Array.prototype.slice.call(arguments);
     this.reenable();
-    this.reenabled = true
+    this.reenabled = true;
     this.run();
   }
 
@@ -420,7 +420,9 @@ class Task extends EventEmitter {
 
   _getParams() {
     if (!this.action) return "";
-    let params = (new RegExp('(?:'+this.action.name+'\\s*|^)\\s*\\((.*?)\\)').exec(this.action.toString().replace(/\n/g, '')) || [,''])[1].replace(/\/\*.*?\*\//g, '').replace(/ /g, '');
+    console.log(this.action.name, this.action.toString());
+    let params = (new RegExp('(?:'+this.action.name+'\\s*|^)\\s*\\((.*?)\\)').exec(this.action.toString().replace(/\n/g, '')) || [''])[1].replace(/\/\*.*?\*\//g, '').replace(/ /g, '');
+    console.log(params);
     return params;
   }
 

lib/utils/file.js

@@ -28,9 +28,9 @@ let fileUtils = new (function () {
 
   // Recursively copy files and directories
   let _copyFile = function (fromPath, toPath, opts) {
-    let from = path.normalize(fromPath)
-    let to = path.normalize(toPath)
-    let options = opts || {}
+    let from = path.normalize(fromPath);
+    let to = path.normalize(toPath);
+    let options = opts || {};
     let fromStat;
     let toStat;
     let destExists;

test/integration/concurrent.js

@@ -9,27 +9,27 @@ suite('concurrent', function () {
   this.timeout(7000);
 
   test(' simple concurrent prerequisites 1', function () {
-    let out = exec(`${JAKE_CMD} -q concurrent:simple1`).toString().trim()
+    let out = exec(`${JAKE_CMD} -q concurrent:simple1`).toString().trim();
     assert.equal('Started A\nStarted B\nFinished B\nFinished A', out);
   });
 
   test(' simple concurrent prerequisites 2', function () {
-    let out = exec(`${JAKE_CMD} -q concurrent:simple2`).toString().trim()
+    let out = exec(`${JAKE_CMD} -q concurrent:simple2`).toString().trim();
     assert.equal('Started C\nStarted D\nFinished C\nFinished D', out);
   });
 
   test(' sequential concurrent prerequisites', function () {
-    let out = exec(`${JAKE_CMD} -q concurrent:seqconcurrent`).toString().trim()
+    let out = exec(`${JAKE_CMD} -q concurrent:seqconcurrent`).toString().trim();
     assert.equal('Started A\nStarted B\nFinished B\nFinished A\nStarted C\nStarted D\nFinished C\nFinished D', out);
   });
 
   test(' concurrent concurrent prerequisites', function () {
-    let out = exec(`${JAKE_CMD} -q concurrent:concurrentconcurrent`).toString().trim()
+    let out = exec(`${JAKE_CMD} -q concurrent:concurrentconcurrent`).toString().trim();
     assert.equal('Started A\nStarted B\nStarted C\nStarted D\nFinished B\nFinished C\nFinished A\nFinished D', out);
   });
 
   test(' concurrent prerequisites with subdependency', function () {
-    let out = exec(`${JAKE_CMD} -q concurrent:subdep`).toString().trim()
+    let out = exec(`${JAKE_CMD} -q concurrent:subdep`).toString().trim();
     assert.equal('Started A\nFinished A\nStarted Ba\nFinished Ba', out);
   });
 

test/integration/jakefile.js

@@ -14,7 +14,7 @@ task('noActionNoPrereqs');
 
 desc('Top-level zerbofrangazoomy task');
 task('zerbofrangazoomy', function () {
-  console.log('Whaaaaaaaa? Ran the zerbofrangazoomy task!')
+  console.log('Whaaaaaaaa? Ran the zerbofrangazoomy task!');
 });
 
 desc('Task that throws');
@@ -237,7 +237,7 @@ namespace('fileTest', function () {
   desc('File task, run if the prereq file changes');
   task('touch-prereq', function () {
     fs.writeFileSync('foo/prereq.txt', 'UPDATED');
-  })
+  });
 
   desc('File task, has a preexisting file (with no associated task) as a prereq');
   file('foo/from-prereq.txt', ['fileTest:foo', 'foo/prereq.txt'], function () {
@@ -332,6 +332,6 @@ namespace("large", function () {
 
   desc("Task with a large number of different prereqs");
   task("different", different, { concurrency: 2 } , function () {
-    console.log("large:different")
-  })
+    console.log("large:different");
+  });
 });

test/integration/jakelib/publish.jake.js

@@ -31,12 +31,12 @@ publishTask('zerb', function () {
     , 'tmp_publish/**'
   ]);
   this.publishCmd = 'node -p -e "\'%filename\'"';
-  this.gitCmd = 'echo'
+  this.gitCmd = 'echo';
   this.scheduleDelay = 0;
 
   this._ensureRepoClean = function () {};
   this._getCurrentBranch = function () {
-    return 'v0.0'
+    return 'v0.0';
   };
 });
 

test/integration/selfdep.js

@@ -24,7 +24,7 @@ suite('selfDep', function () {
       exec(`${JAKE_CMD} selfdepconst`);
     }
     catch(e) {
-      assert(e.message.indexOf('dependency of itself') > -1)
+      assert(e.message.indexOf('dependency of itself') > -1);
     }
   });
 
@@ -33,7 +33,7 @@ suite('selfDep', function () {
       exec(`${JAKE_CMD} selfdepdyn`);
     }
     catch(e) {
-      assert(e.message.indexOf('dependency of itself') > -1)
+      assert(e.message.indexOf('dependency of itself') > -1);
     }
   });