Merge pull request #26 from atom-ide-community/fix/dependency-installation

fix: #25

Author: appelgriebsch

lib/datatip-manager.js

@@ -11,17 +11,17 @@ module.exports = class DatatipManager {
      * [subscriptions description]
      * @type {CompositeDisposable}
      */
-    this.subscriptions = null;
+    this.subscriptions = new CompositeDisposable();
     /**
      * [providerRegistry description]
      * @type {ProviderRegistry}
      */
-    this.providerRegistry = null;
+    this.providerRegistry = new ProviderRegistry();
     /**
      * [watchedEditors description]
      * @type {Array<TextEditor>}
      */
-    this.watchedEditors = null;
+    this.watchedEditors = new WeakSet();
     /**
      * [editor description]
      * @type {TextEditor}
@@ -79,16 +79,17 @@ module.exports = class DatatipManager {
      */
     this.cursorMoveTimer = null;
 
+    /**
+     * [renderer description]
+     * @type {[type]}
+     */
     this.renderer = null;
   }
 
   /**
    * [initialize description]
    */
   initialize(renderer) {
-    this.subscriptions = new CompositeDisposable();
-    this.providerRegistry = new ProviderRegistry();
-    this.watchedEditors = new WeakSet();
     this.renderer = renderer;
 
     this.subscriptions.add(

lib/main.js

@@ -49,11 +49,8 @@ module.exports = {
   },
 
   consumeMarkdownRenderer(renderer) {
-    // kick it off after installing the package dependencies
-    require('atom-package-deps').install('atom-ide-datatip')
-      .then(() => {
-        console.log('All dependencies installed, good to go')
-        this.datatipManager.initialize(renderer);
-      });
+    require('atom-package-deps').install('atom-ide-datatip').then(() => {
+      this.datatipManager.initialize(renderer);
+    });
   }
 };