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.

Sign up for GitHub

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

Jump to bottom

feat(cli): add --dry-run flag for important commands #416

Open
merklefruit opened this issue Nov 22, 2024 · 2 comments · May be fixed by #756
Open

feat(cli): add --dry-run flag for important commands #416

merklefruit opened this issue Nov 22, 2024 · 2 comments · May be fixed by #756
Labels
C: bolt-cli Component: bolt CLI T: feature Type: Feature
Milestone
v0.4.1

Comments

@merklefruit
Copy link
Collaborator

We should add a --dry-run flag to all CLI commands that require an RPC connection.

When this is specified, we should use an Anvil chain fork as blockchain provider.
This will be very handy for people looking to register validators or opt-in to various contracts.
@merklefruit merklefruit added C: smart-contracts Component: smart contracts T: feature Type: Feature labels Nov 22, 2024
@merklefruit merklefruit added this to the v0.4: Holesky 2 milestone Nov 22, 2024
@merklefruit merklefruit added C: bolt-cli Component: bolt CLI and removed C: smart-contracts Component: smart contracts labels Nov 22, 2024
@mempirate mempirate modified the milestones: v0.4: Holesky 2, v0.3.2 Nov 27, 2024
@0ex-d
Copy link
Contributor

0ex-d commented Jan 19, 2025

Totally agree, a few more RPC-less tests 🚀. would get a PR rolling for this soon. Happy weekend guys!

@merklefruit
Copy link
Collaborator Author

Hi @0ex-d, I think the Foundry abstraction here could help a lot!

By default running a command without the --broadcast flag will only simulate the command locally to give user assurances that the command would go through, and then they can use that flag to actually send the transaction when they're sure

sauce https://book.getfoundry.sh/tutorials/solidity-scripting#high-level-overview

@0ex-d 0ex-d mentioned this issue Jan 22, 2025
Closed
@0ex-d 0ex-d linked a pull request Jan 31, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: bolt-cli Component: bolt CLI T: feature Type: Feature
Projects
None yet
Milestone
v0.4.1
3 participants
@0ex-d @mempirate @merklefruit