nippy icon indicating copy to clipboard operation
nippy copied to clipboard

Reducing the dependency footprint

Open scarytom opened this issue 1 year ago • 2 comments
trafficstars

I'm using nippy without any compression. It would be nice to exclude the dependencies that are (I think) only required for compression, and keep the dependency footprint smaller. At the moment, if I exclude these dependencies, then compilation fails. I guess the code could be re-worked to dynamically reference classes in these libs? Happy to experiment with this and do a pull request if you think it is acceptable to do this.

https://github.com/taoensso/nippy/blob/5a9a3913319bde692fb2a50696036b5493e15eb1/project.clj#L13-L14

scarytom avatar Jun 06 '24 12:06 scarytom

@scarytom Hi Tom! Would be happy to look at a PR that makes these optional if it's pretty simple and doesn't negatively impact current users 👍

ptaoussanis avatar Jun 06 '24 12:06 ptaoussanis

I quickly put a pull-request together, just to show you what it would look like before refining anything too much. I'm concerned that these changes might affect performance, which is obviously a big deal, so benchmarking definitely required.

scarytom avatar Jun 06 '24 14:06 scarytom