search

mirage / irmin-server

A high-performance server for Irmin
ISC License
24 stars 7 forks source link

Simplify client interface #57

Closed zshipko closed 2 years ago

zshipko commented 2 years ago

This PR merges Irmin_client.Store.Make and Irmin_client.Make to avoid having to maintain too many overlapping commands. Some of the Store commands may be kept in order to optimize certain functions (find/get/set/remove, ...)

Fixes #56

zshipko commented 2 years ago

@patricoferris @dinakajoy I hope to merge this PR early next week, it touches pretty much every part of the codebase so will likely cause issues if you have an in-progress changes. I can wait to merge if you have any PRs you're planning, but am also happy to help adapt your projects to work with these changes.

Essentially this PR moves Client.Store.Make to Client.Make and merges the two APIs since there was a lot of duplicate functionality.