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
respect silent argument in script command #4160
respect silent argument in script command #4160
Conversation
03779e3
to
e1ffba8
Compare
e1ffba8
to
d69efd2
Compare
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.
the command is still using regular println!
, right?
we'd need to refactor them as well and go trough shell
842650b
to
67835b1
Compare
67835b1
to
755fed5
Compare
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.
few nits, otherwise lgtm
cli/src/cmd/forge/script/sequence.rs
Outdated
|
||
println!("\nTransactions saved to: {path}\n"); | ||
shell::println(format!("\nTransactionsdffs saved to: {path}\n"))?; | ||
// println!("\nTransactions saved to: {path}\n"); |
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.
// println!("\nTransactions saved to: {path}\n"); |
cli/src/cmd/forge/script/cmd.rs
Outdated
if self.opts.args.silent { | ||
shell::set_shell(shell::Shell::from_args(true, false))?; |
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.
can be combined by using self.opts.args.silent
as argument
the failing script tests indicate that something's not behaving as expected |
c5f80b3
to
09b46eb
Compare
Motivation
Respect --silent argument in forge script command
Solution
Fixes #4009