Update dependencies, reformat code and change target node from 4 to 8+

[armano2]

• Update dependencies
• reformat code
• change target node from 4 to 8+
• fix unit tests for CLI
• drop babel build - no longer needed for node 8-13
• drop gulp in favor of native cli commands
• add ISC LICENSE.md based on package.json
• remove data folder and use dictionary-* packages instead (we could add automatic suggestion/loading of other langagues if needed)
• drop hunspell-spellchecker in favor of way faster nodehun
  • i can change it to nspell if js library is preferred
  • hunspell-spellchecker seems to be no longer maintained
• drop async library in favor of native promises

---

small note: (node 8)

• tests are executed in ~1sec on this branch
• tests are executed in ~6sec on master

this is not actual benchmark but validation is way faster

@robatwilliams @lukeapage

[armano2]

Update: Tests now are executed without exceptions found on master and some of them are fixed to actually do stuff instead of being broken due to mix between sync/async calls)

https://travis-ci.org/lukeapage/node-markdown-spellcheck/jobs/593726326#L283

read more

``` cli interactive ✓ should work with no mistakes (105ms) ✓ should work with a single ignore (node:5062) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'action' of undefined at /home/travis/build/lukeapage/node-markdown-spellcheck/es5/cli-interactive.js:114:20 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1) (node:5062) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code. (node:5062) UnhandledPromiseRejectionWarning: AssertionError: expected false to equal true at /home/travis/build/lukeapage/node-markdown-spellcheck/test/cli-interactive.js:83:43 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2) ✓ correct word with 2 words (node:5062) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'action' of undefined at /home/travis/build/lukeapage/node-markdown-spellcheck/es5/cli-interactive.js:114:20 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3) (node:5062) UnhandledPromiseRejectionWarning: AssertionError: expected false to equal true at /home/travis/build/lukeapage/node-markdown-spellcheck/test/cli-interactive.js:101:53 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 4) ✓ correct word with incorrect word (node:5062) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'action' of undefined at /home/travis/build/lukeapage/node-markdown-spellcheck/es5/cli-interactive.js:114:20 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 5) (node:5062) UnhandledPromiseRejectionWarning: AssertionError: expected false to equal true at /home/travis/build/lukeapage/node-markdown-spellcheck/test/cli-interactive.js:130:52 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 6) ✓ correct word with filtered word (node:5062) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'action' of undefined at /home/travis/build/lukeapage/node-markdown-spellcheck/es5/cli-interactive.js:114:20 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 7) (node:5062) UnhandledPromiseRejectionWarning: AssertionError: expected false to equal true at /home/travis/build/lukeapage/node-markdown-spellcheck/test/cli-interactive.js:156:48 at at process._tickDomainCallback (internal/process/next_tick.js:229:7) (node:5062) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 8) ```

[JaKXz]

This is awesome! @lukeapage sorry to ping on an old PR but I'd say this is a worthy PR for v2.0?

[armano2]

due to lack of interest in this PR, i'm closing this, i moved away from this library to cspell