Remove print to console for named proofs in nargo prove
#718
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related issue(s)
Resolves #702
Description
Summary of changes
I've reworked
prove_with_path
so that it either prints to console or writes to file based on whether it has been passed a name for the proof its generating.After this PR, the output of
nargo prove
is as followsWe do end up in a bit of an awkward situation where
prove_with_path
takes an option as both an argument and its return value. This happens as it's a bit of a "do everything" function, my preference is to push some of the printing/IO further up into more CLI specific code and implement #626 which would simplify it greatly, but that can happen later.Dependency additions / changes
N/A
Test additions / changes
N/A
Checklist
cargo fmt
with default settings.Additional context