-
Notifications
You must be signed in to change notification settings - Fork 405
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
workspace tooling overhaul #54
Conversation
Development cli usage # from the workspace
cargo install --path tools/tools-dev --force # to override previous installation List of all commands:
The cli is structured in a way that:
|
|
CLI usage: Basics:cargo nexus new # outside of workspace, otherwise cargo treats crate as its member
cargo run
cargo nexus run [-v] # practically the same, can print trace Local provecargo nexus prove local # --release to run in release, --bin to specify the binary That's it, the command will compile configurations and generate Network prove# in a separate terminal, can run from anywere, e.g. $HOME
cargo nexus node
# another terminal
cargo nexus node -p # pcd node Once they're done reading PP file: cargo nexus prove
# you've got something like <HASH> 0 0 incomplete When you see
VerifyNot supported, prover crate doesn't know how to decode proofs (yet) |
This looks awesome. Three suggestions:
|
These are great points. 1) is important and I'll fix it in this PR, although I believe profile handling should be reworked. |
Shouldn't release mode then be the default, and debug mode the one you have to specify with --debug if you want it? |
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.
Looks good. Just a few minor questions.
Yes I think this is a better idea |
I'd like to avoid making this change in this PR, especially because we had a discussion about our own Rust target |
No description provided.