search

aantron / promise

Light and type-safe binding to JS promises
MIT License
340 stars 24 forks source link

Use Belt everywhere. #58

Closed cknitt closed 4 years ago

cknitt commented 4 years ago

As discussed in #57, make sure that on the JS side we import only belt_Array.js and not caml_array.js, too.

aantron commented 4 years ago

Thanks! This decreased the incremental (that is, compared to someone already using Belt) compiled size from 3913 bytes to 1184 bytes.

aantron commented 4 years ago

Travis CI status reporting seems to be broken for PRs at the moment. There are several bug reports like this one at Travis with no satisfactory solution yet. The builds are running, though, e.g. https://travis-ci.org/github/aantron/promise/builds/700295405 ran for this PR.