-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat(cli): Support --anvil.*
flags
#1358
Conversation
|
--anvil.*
notation for Anvil flags
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.
would be really cool if we could progmatically just pass any arguments based on if they start with --anvil.
prefix rather than having to code them here (that way if they add/change arguments in the future, it always works with user's setup)
I get what you’re saying and I tend to agree but I found two reasons to keep them in the code: 1- they’re used in the docs on the website, 2- if someone mistypes a flag, commander suggests the right one |
--anvil.*
notation for Anvil flags--anvil.*
flags
This PR makes a clearer distinction between Cannon and Anvil flags by prefixing Anvil flags with --anvil.*, improving clarity and making their origin explicit. It also includes the upgrade to the latest version of Commander.
Closes https://linear.app/usecannon/issue/CAN-508/add-foundry-params-to-all-cannon-commands-with-forge-and-anvil