rrdelaney / reason-scripts

:beginner: Create a ReasonML and React development environment
MIT License
628 stars 58 forks source link

added bs-fetch to bs-dependencies #53

Closed shrynx closed 6 years ago

shrynx commented 6 years ago

added bs-fetch as a dependency as fetch is already added as a polyfill. https://github.com/reasonml-community/reason-scripts/blob/master/config/polyfills.js#L20

rrdelaney commented 6 years ago

Omg yes :heart: Thanks!!!

shrynx commented 6 years ago

sorry it is failing, i was doing it half asleep, will fit it soon.

rrdelaney commented 6 years ago

I just fixed up some merge conflicts from the v0.9 branch :smile:

shrynx commented 6 years ago

Oh nice thanks a lot 😄

rrdelaney commented 6 years ago

Also, don't worry about the tests, they've been broken for a while.