Merge pull request #14 from azuqua/kevin/develop

remove TODOs from CLI and update README with patch
azuqua · Mar 30, 2017 · 20244e4 · 20244e4
2 parents bbfda4a + 0f37d53
commit 20244e4
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -458,6 +458,8 @@ From here, you can reference the documentation found on the github pages for the
 
 ### <a name="ChangeLog"></a>ChangeLog
 
+- 1.1.2:
+  - Remove TODOs from bin/cli
 - 1.1.1:
   - Grammatical fixes to README
 - 1.1.0:

diff --git a/bin/cli.js b/bin/cli.js
@@ -84,7 +84,7 @@ class Client extends EventEmitter {
       });
       return this;
     }
-    cb(new Error("TODO"));
+    cb(new Error("Disconnected from server."));
     return this;
   }
 
@@ -103,7 +103,7 @@ class Client extends EventEmitter {
     }
     this._connected = false;
     this._rcv.flush().forEach((el) => {
-      el.cb(new Error("TODO"));
+      el.cb(new Error("Disconnected from server."));
     });
     this.emit("disconnect");
     return this;