Open evanshortiss opened 6 years ago
Build is failing on node8. Let's remove it. I will need time to get that tested with RainCatcher and I think we can merge it then.
@evanshortiss Reviewed locally. Great contribution. Some minor problems need to be resolved and we are good to go!
@wtrocki updated to remove Node.js 0.10 and 8 (temporarily) from CI 👍
@evanshortiss Thanks for contribution. Tried that locally
Added 2 commits to fix minor issues. If you are ok with all I think we can merge it and publish your changes to npm.
@wtrocki looks good, the only thing is I might remove ts-node
as per the comments you and @MikeyBurkman had. Safer that way.
Triggering build again. It's failing due to npm issues.
See notes in the PR for more information. This is aimed at making some improvements to TS support/docs/typings.