Skip to content
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

Refactor library and binary crate #67

Merged
merged 2 commits into from
Jan 31, 2023
Merged

Refactor library and binary crate #67

merged 2 commits into from
Jan 31, 2023

Conversation

ralexstokes
Copy link
Owner

the previous repo had a library crate containing common code mixed with a "builder" crate

first change: refactor the common crate as mev-lib from the distinct builder crate

second change: move the command line utility from the top-level mev-rs crate into a separate binary crate

now that the work has been done I see I could have just left the mev-rs crate alone (I was thinking mev-lib would become mev-rs but the names will clash b/t binary crate and library crate) but the work is already done and this does lay the foundation for future command line utilities in the mev-rs ecosystem that could be distinct from running the various node types (e.g. data utilities?)

@ralexstokes ralexstokes merged commit c3ef593 into main Jan 31, 2023
@ralexstokes ralexstokes deleted the repo-refactor branch January 31, 2023 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant