Remove is-observable due to issues with RxJS@6

SamVerschueren · May 9, 2018 · bee2889 · bee2889
1 parent 447e8d1
commit bee2889
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/utils.js b/lib/utils.js
@@ -1,6 +1,8 @@
 'use strict';
 const isStream = require('is-stream');
-const isObservable = require('is-observable');
+
+// TODO https://github.com/sindresorhus/is-observable/issues/1
+const isObservable = obj => Boolean(obj && typeof obj.subscribe === 'function' && obj.constructor.name === 'Observable');
 
 exports.isListr = obj => Boolean(obj && obj.setRenderer && obj.add && obj.run);
 exports.isObservable = obj => isObservable(obj);

diff --git a/package.json b/package.json
@@ -42,7 +42,6 @@
     "cli-truncate": "^0.2.1",
     "figures": "^1.7.0",
     "indent-string": "^2.1.0",
-    "is-observable": "^0.2.0",
     "is-promise": "^2.1.0",
     "is-stream": "^1.1.0",
     "listr-silent-renderer": "^1.1.1",