tweag / cooked-validators

MIT License
39 stars 11 forks source link

Trim dependencies for better integration with dApp codebases #69

Closed kk-hainq closed 7 months ago

kk-hainq commented 2 years ago

Is your feature request related to a problem? Please describe. I'd like to integrate this library into our codebase but the dependencies according to the current plutus-apps release is quite heavy-weight and easy to conflict with newer dependencies.

Describe the solution you'd like Aggressively trimming dependencies on the next plutus-apps release would save a lot of resources building this library for both maintainers and library users. Fewer dependencies mean fewer conflicting risks integrating the library into dApp codebases as well.

Describe alternatives you've considered Having to revert existing dependencies and API breaking change fixes, and add unused dependencies to integrate this library into our dApp codebase.

Additional context Upstream is already moving well on this end:

I'd love to open a PR to do this after the next plutus-apps release!

VictorCMiraldo commented 2 years ago

I 100% agree with that. We'd be happy to accept a PR too! Thanks! :)

kk-hainq commented 2 years ago

Can we trade merged PRs for future discounts, Victor? :cry:

VictorCMiraldo commented 2 years ago

I wish I could have a say in that, but that's way above my pay-grade, unfortunately! :lol:

mmontin commented 7 months ago

Closing because this was handled when moving to cooked v2.