Remove unneeded semicolons

Author: Ian VanSchooten

lib/es5-helpers.js

@@ -1,4 +1,4 @@
-'use strict';
+'use strict'
 
 const ChildProcess = require('child_process')
 const Path = require('path')
@@ -7,7 +7,7 @@ const FS = require('fs')
 let prefixPath = null
 
 function find(startDir, names) {
-  let localNames;
+  let localNames
   if (typeof names === 'string') {
     localNames = [names]
   } else {

lib/main.js

@@ -80,11 +80,11 @@ export default {
         if (!textEditor || textEditor.isModified()) {
           // Ignore invalid or unsaved text editors
           atom.notifications.addError('Linter-ESLint: Please save before fixing')
-          return;
+          return
         }
 
         if (this.worker === null) {
-          return;
+          return
         }
 
         this.worker.request('FIX', {
@@ -135,7 +135,7 @@ export default {
       lint: textEditor => {
         const text = textEditor.getText()
         if (text.length === 0) {
-          return Promise.resolve([]);
+          return Promise.resolve([])
         }
         const filePath = textEditor.getPath()
         const fileDir = Path.dirname(filePath)
@@ -147,7 +147,7 @@ export default {
             type: 'Info',
             text: 'Worker initialization is delayed. Please try saving or typing to begin linting.',
             range: Helpers.rangeFromLineNumber(textEditor, 0)
-          }]);
+          }])
         }
 
         return this.worker.request('JOB', {

lib/worker.js

@@ -42,7 +42,7 @@ Communication.on('JOB', function(job) {
   // Determine which eslint instance to use
   const eslintNewPath = findEslintDir(params)
   if (eslintNewPath !== eslintPath) {
-    eslint = getEslintCli(eslintNewPath);
+    eslint = getEslintCli(eslintNewPath)
     eslintPath = eslintNewPath
   }
 

package.json

@@ -43,7 +43,8 @@
     "rules": {
       "no-empty": 0,
       "no-console": 0,
-      "no-new": 0
+      "no-new": 0,
+      "semi": [2, "never"]
     },
     "parser": "babel-eslint",
     "globals": {

spec/es5-helpers-spec.js

@@ -1,4 +1,4 @@
-'use babel';
+'use babel'
 
 import Path from 'path'
 import OS from 'os'
@@ -7,7 +7,7 @@ import {
   determineConfigFile,
   findEslintDir,
   getEslintCli
-} from '../lib/es5-helpers';
+} from '../lib/es5-helpers'
 
 let fixtureDir
 
@@ -134,7 +134,7 @@ describe('The es5 linter-eslint helper', () => {
           fileDir: dir,
           global: false,
         }
-        const foundEslintDir = findEslintDir(params);
+        const foundEslintDir = findEslintDir(params)
         expect(foundEslintDir).toEqual(expectedPath)
       })
     })
@@ -151,6 +151,6 @@ describe('The es5 linter-eslint helper', () => {
 
   it('teardown', () => {
     // Clean up temporary directory
-    rm('-r', fixtureDir);
+    rm('-r', fixtureDir)
   })
 })

spec/linter-eslint-spec.js

@@ -1,77 +1,77 @@
-'use babel';
+'use babel'
 
 describe('The eslint provider for Linter', () => {
   const {spawnWorker} = require('../lib/helpers')
   const worker = spawnWorker()
-  const lint = require('../lib/main').provideLinter.call(worker).lint;
+  const lint = require('../lib/main').provideLinter.call(worker).lint
 
   beforeEach(() => {
     waitsForPromise(() => {
       return atom.packages.activatePackage('language-javascript').then(() =>
         atom.workspace.open(__dirname + '/fixtures/files/good.js')
       )
-    });
-  });
+    })
+  })
 
   describe('checks bad.js and', () => {
-    let editor = null;
+    let editor = null
     beforeEach(() => {
       waitsForPromise(() => {
-        atom.config.set('linter-eslint.disableEslintIgnore', true);
+        atom.config.set('linter-eslint.disableEslintIgnore', true)
         return atom.workspace.open(__dirname + '/fixtures/files/bad.js').then(openEditor => {
-          editor = openEditor;
-        });
-      });
-    });
+          editor = openEditor
+        })
+      })
+    })
 
     it('finds at least one message', () => {
       return lint(editor).then(messages => {
-        expect(messages.length).toBeGreaterThan(0);
-      });
-    });
+        expect(messages.length).toBeGreaterThan(0)
+      })
+    })
 
     it('verifies that message', () => {
       return lint(editor).then(messages => {
-        expect(messages[0].type).toBeDefined();
-        expect(messages[0].type).toEqual('Error');
-        expect(messages[0].html).not.toBeDefined();
-        expect(messages[0].text).toBeDefined();
-        expect(messages[0].text).toEqual('"foo" is not defined.');
-        expect(messages[0].filePath).toBeDefined();
-        expect(messages[0].filePath).toMatch(/.+spec[\\\/]fixtures[\\\/]files[\\\/]bad\.js$/);
-        expect(messages[0].range).toBeDefined();
-        expect(messages[0].range.length).toEqual(2);
-        expect(messages[0].range).toEqual([[0, 0], [0, 9]]);
-      });
-    });
-  });
+        expect(messages[0].type).toBeDefined()
+        expect(messages[0].type).toEqual('Error')
+        expect(messages[0].html).not.toBeDefined()
+        expect(messages[0].text).toBeDefined()
+        expect(messages[0].text).toEqual('"foo" is not defined.')
+        expect(messages[0].filePath).toBeDefined()
+        expect(messages[0].filePath).toMatch(/.+spec[\\\/]fixtures[\\\/]files[\\\/]bad\.js$/)
+        expect(messages[0].range).toBeDefined()
+        expect(messages[0].range.length).toEqual(2)
+        expect(messages[0].range).toEqual([[0, 0], [0, 9]])
+      })
+    })
+  })
 
   it('finds nothing wrong with an empty file', () => {
     waitsForPromise(() => {
       return atom.workspace.open(__dirname + '/fixtures/files/empty.js').then(editor => {
         return lint(editor).then(messages => {
-          expect(messages.length).toEqual(0);
-        });
-      });
+          expect(messages.length).toEqual(0)
+        })
+      })
     })
-  });
+  })
 
   it('finds nothing wrong with a valid file', () => {
     waitsForPromise(() => {
       return atom.workspace.open(__dirname + '/fixtures/files/good.js').then(editor => {
         return lint(editor).then(messages => {
-          expect(messages.length).toEqual(0);
-        });
-      });
+          expect(messages.length).toEqual(0)
+        })
+      })
     })
-  });
+  })
 
   describe('when resolving import paths using eslint-plugin-import', () => {
     it('correctly resolves imports from parent', () => {
       waitsForPromise(() => {
         return atom.workspace.open(`${__dirname}/fixtures/import-resolution/nested/importing.js`).then(editor => {
           return lint(editor).then(messages => {
-            expect(messages.length).toEqual(0);
+            expect(messages.length).toEqual(0)
           })
         })
       })
@@ -83,10 +83,10 @@ describe('The eslint provider for Linter', () => {
       waitsForPromise(() => {
         return atom.workspace.open(`${__dirname}/fixtures/eslintignore/ignored.js`).then(editor => {
           return lint(editor).then(messages => {
-            expect(messages.length).toEqual(0);
+            expect(messages.length).toEqual(0)
           })
         })
       })
     })
   })
-});
+})