Merge pull request #17 from AtomLinter/arcanemagus/improve-activation

Defer package dependency checks
AtomLinter · Jul 13, 2017 · 33e898b · 33e898b
2 parents 5e75060 + 15f0db4
commit 33e898b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/lib/minimap-linter.js b/lib/minimap-linter.js
@@ -9,11 +9,22 @@ const bindings = new Map();
 export default {
   // Atom package lifecycle events start
   activate() {
-    require('atom-package-deps').install('minimap-linter');
+    this.idleCallbacks = new Set();
+    let depsCallbackID;
+    const installMinimapLinterDeps = () => {
+      this.idleCallbacks.delete(depsCallbackID);
+      if (!atom.inSpecMode()) {
+        require('atom-package-deps').install('minimap-linter');
+      }
+    };
+    depsCallbackID = window.requestIdleCallback(installMinimapLinterDeps);
+    this.idleCallbacks.add(depsCallbackID);
     this.subscriptions = new CompositeDisposable();
   },
 
   deactivate() {
+    this.idleCallbacks.forEach(callbackID => window.cancelIdleCallback(callbackID));
+    this.idleCallbacks.clear();
     if (!this.minimapProvider) {
       return;
     }

diff --git a/package.json b/package.json
@@ -71,7 +71,8 @@
       "atom": true
     },
     "env": {
-      "node": true
+      "node": true,
+      "browser": true
     }
   }
 }