Refactor promises

[benjamingr]

Refactor promise usage in the code - removing the resolvable() function and returning from fetch which already returns a promise.

I wonder if this helps with the file size :)

Fixes #10

[mikeal]

Travis is broken right now cause of a config issue I can't resolve until I get the npm name, are all the tests passing?

[benjamingr]

Yes, at least on my machine. I wouldn't submit a PR otherwise without saying so.

This is not urgent - it's a code style fix so there is no rush about it.

Let me know if there is any help I can offer with the CI - I'm generally bad with CIs so I try to pick up opportunities to learn.

[benjamingr]

Hey @mikeal are you still looking into integrating this? Let me know if I should resolve the conflicts and continue work. Again - only if this works out for you and feel free to say you don't have time/motivation to work on this and I'll drop it with no hard feelings <3

[mikeal]

yes, i would love to merge this :)

[ewolfe]

@benjamingr thanks for working on this, would you mind fixing the conflicting files?

[benjamingr]

Sorry, forgot about this - just did the change again and force pushed.

[ewolfe]

Looks like this is good to go @mikeal

[mikeal]

🎉 This PR is included in version 2.0.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀