search

infinitered / apisauce

Axios + standardized errors + request/response transforms.
MIT License
2.78k stars 184 forks source link

Removing Ramda (from prod build) and going from 19k to 6k #281

Closed jamonholmgren closed 2 years ago

jamonholmgren commented 2 years ago

This PR removes Ramda from our production build of apisauce. (It's still present in devDependencies because I didn't feel like untangling the tests.)

            commonjs    umd
With Ramda:     7.6k  19.2k
Without:        6.6k   5.8k

As you can see, the size win is significant -- 1k less for commonjs and 13k less for umd (not sure why umd has such a huge win, but i'll take it).

At some point it would be nice to remove Ramda from the tests too, but that's a chore.

infinitered-circleci commented 2 years ago

:tada: This PR is included in version 2.1.4 :tada:

The release is available on:

Your semantic-release bot :package::rocket: