-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extract persistence related code to a new shared crate #1494
Conversation
da13e6d
to
d846961
Compare
d846961
to
38274e7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
I left few comments for typos below. Also, should we bump the minor version of mithril-common
as we changed the available features in the crate?
Yup, bumping the minor version of |
Add new crate to: * global Makefile * docsite reference table * `ci.yml` to `cargo-doc` generation step * `flake.nix` builded artifacts
38274e7
to
f2aedf1
Compare
f2aedf1
to
e029a20
Compare
e029a20
to
535ba5a
Compare
Content
This PR create a new shared crate,
mithril-persistence
. All all store & database related code frommithril-common
has been moved to it.This makes enhance our code base maintainability, reduce the dependencies of the mithril-common crate, allow to remove its
database
feature.Pre-submit checklist
Issue(s)
Closes #1492.