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

Reorganize Cast to expose the cli logic as pub #8421

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ewoolsey
Copy link
Contributor

Motivation

I'm writing a cli tool and I would like to be able to run cast programatically. Currently this isn't possible, as the logic to run the cli is hidden inside /bin.

Solution

I've made the project structure of cast similar to that of anvil which exposes it's internals more liberally. Currently I have this:

pub mod cmd;
pub mod opts;
pub mod tx;

which may be exposing more than we want/need. I would like some feedback on exactly what we should/could be making public. Thanks!

ArkFunds

This comment was marked as spam.

@DaniPopes
Copy link
Member

What exactly do you need from the binary crate? Making CLIs private was an intentional choice

@zerosnacks zerosnacks modified the milestone: v1.0.0 Jul 31, 2024
@zerosnacks zerosnacks added the C-cast Command: cast label Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-cast Command: cast
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants