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

Suggestions for 813-standardize-cli #61

Merged
merged 16 commits into from
Jun 25, 2024

Conversation

rrybarczyk
Copy link

No description provided.

@rrybarczyk rrybarczyk marked this pull request as draft June 11, 2024 13:20
@plebhash plebhash force-pushed the 813-standardize-cli branch from a677984 to 16f08e6 Compare June 12, 2024 14:04
@plebhash
Copy link
Owner

we should add a new app-helpers crate to roles-utils with:

  • module for arg parsing
  • module for custom config deserializers
  • module for parsing coinbase output

@plebhash
Copy link
Owner

plebhash commented Jun 21, 2024

@rrybarczyk FYI some PRs that could affect this one when they get merged into upstream dev:

I'll keep rebasing 813-standardize-cli with those changes as they get merged

@plebhash plebhash marked this pull request as ready for review June 25, 2024 11:53
@plebhash plebhash merged commit acb4412 into plebhash:813-standardize-cli Jun 25, 2024
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.

2 participants