🔥 Remove timeout on worker spawn

Author: steelbrain

lib/main.js

@@ -99,12 +99,6 @@ exports.default = {
           return;
         }
 
-        if (_this.worker === null) {
-          // Abort if worker is not yet ready
-          atom.notifications.addError('Linter-ESLint: Not ready, please try again');
-          return;
-        }
-
         _this.worker.request('FIX', {
           fileDir: fileDir,
           filePath: filePath,
@@ -119,10 +113,6 @@ exports.default = {
       }
     }));
 
-    // Reason: I (steelbrain) have observed that if we spawn a
-    // process while atom is starting up, it can increase startup
-    // time by several seconds, But if we do this after 5 seconds,
-    // we barely feel a thing.
     const initializeWorker = function () {
       if (_this.active) {
         var _spawnWorker = (0, _helpers.spawnWorker)();
@@ -142,7 +132,7 @@ exports.default = {
         });
       }
     };
-    setTimeout(initializeWorker, 5 * 1000);
+    initializeWorker();
   },
   deactivate: function () {
     this.active = false;
@@ -166,15 +156,6 @@ exports.default = {
         const fileDir = _path2.default.dirname(filePath);
         const showRule = atom.config.get('linter-eslint.showRuleIdInMessage');
 
-        if (_this2.worker === null) {
-          return Promise.resolve([{
-            filePath: filePath,
-            type: 'Info',
-            text: 'Worker initialization is delayed. Please try saving or typing to begin linting.',
-            range: Helpers.rangeFromLineNumber(textEditor, 0)
-          }]);
-        }
-
         return _this2.worker.request('JOB', {
           fileDir: fileDir,
           filePath: filePath,

src/main.js

@@ -83,12 +83,6 @@ export default {
           return
         }
 
-        if (this.worker === null) {
-          // Abort if worker is not yet ready
-          atom.notifications.addError('Linter-ESLint: Not ready, please try again')
-          return
-        }
-
         this.worker.request('FIX', {
           fileDir: fileDir,
           filePath: filePath,
@@ -103,10 +97,6 @@ export default {
       }
     }))
 
-    // Reason: I (steelbrain) have observed that if we spawn a
-    // process while atom is starting up, it can increase startup
-    // time by several seconds, But if we do this after 5 seconds,
-    // we barely feel a thing.
     const initializeWorker = () => {
       if (this.active) {
         const {child, worker, subscription} = spawnWorker()
@@ -121,7 +111,7 @@ export default {
         })
       }
     }
-    setTimeout(initializeWorker, 5 * 1000)
+    initializeWorker()
   },
   deactivate: function() {
     this.active = false
@@ -143,15 +133,6 @@ export default {
         const fileDir = Path.dirname(filePath)
         const showRule = atom.config.get('linter-eslint.showRuleIdInMessage')
 
-        if (this.worker === null) {
-          return Promise.resolve([{
-            filePath: filePath,
-            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', {
           fileDir: fileDir,
           filePath: filePath,