RationalJS / future

A Js.Promise alternative for ReasonML
214 stars 15 forks source link

Run refmt on all files and replace |. with -> in docs and comments #30

Closed johnhaley81 closed 5 years ago

johnhaley81 commented 5 years ago

This fixes #28

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 50


Totals Coverage Status
Change from base Build 48: 0.0%
Covered Lines: 111
Relevant Lines: 125

💛 - Coveralls
scull7 commented 5 years ago

:shipit: Looks good to me

gilbert commented 5 years ago

There are quite a lot of ->() parenthesis. Are these actually necessary, or are they just precautions inserted by the refmt?

johnhaley81 commented 5 years ago

Good eyes @gilbert. Those aren't needed and I removed them.

gilbert commented 5 years ago

Great, thank you!