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

Add dry run mode to forge create, dumping broadcast to stdout #3356

Closed
nlnw opened this issue Sep 25, 2022 · 1 comment · Fixed by #9420
Closed

Add dry run mode to forge create, dumping broadcast to stdout #3356

nlnw opened this issue Sep 25, 2022 · 1 comment · Fixed by #9420
Assignees
@zerosnacks
Labels
C-forge Command: forge Cmd-forge-create Command: forge create first issue A good way to start contributing P-normal Priority: normal T-feature Type: feature
Milestone
v1.0.0

Comments

@nlnw
Copy link

nlnw commented Sep 25, 2022

Component

Forge

Describe the feature you would like

It would be nice to have a flag for "forge create" that allows it to just output the deployment bytecode without actually sending the transaction. That way you have the option to deploy yourself manually later, or batch the deploy across many chains more easily using a different tool.

Additional context

No response
@nlnw nlnw added the T-feature Type: feature label Sep 25, 2022
@gakonst gakonst added this to Foundry Sep 25, 2022
@gakonst gakonst moved this to Todo in Foundry Sep 25, 2022
@rkrasiuk rkrasiuk added C-forge Command: forge Cmd-forge-create Command: forge create labels Sep 26, 2022
@zerosnacks
Copy link
Member

Would this not be largely achievable by forge inspect $CONTRACT bytecode?

I think it does make sense to add a dry run mode to forge create that would dump this kind of information to stdout

@zerosnacks zerosnacks changed the title Support a flag to output deployment bytecode to disk or console without sending tx Add dry run mode to forge create, dumping broadcast to stdout Jun 26, 2024
@zerosnacks zerosnacks added the P-normal Priority: normal label Jun 26, 2024
@zerosnacks zerosnacks added this to the v1.0.0 milestone Jul 26, 2024
@grandizzy grandizzy added the first issue A good way to start contributing label Oct 17, 2024
@zerosnacks zerosnacks mentioned this issue Nov 27, 2024
Merged
@zerosnacks zerosnacks self-assigned this Nov 27, 2024
@zerosnacks zerosnacks moved this from Todo to In Progress in Foundry Nov 27, 2024
@github-project-automation github-project-automation bot moved this from In Progress to Done in Foundry Nov 27, 2024
@grandizzy grandizzy moved this from Done to Completed in Foundry Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zerosnacks zerosnacks

Labels
C-forge Command: forge Cmd-forge-create Command: forge create first issue A good way to start contributing P-normal Priority: normal T-feature Type: feature
Projects
Foundry
Status: Completed
Milestone
v1.0.0
Development

Successfully merging a pull request may close this issue.

feat: add --broadcast flag to forge create, default to dry run mode foundry-rs/foundry
4 participants
@rkrasiuk @grandizzy @zerosnacks @nlnw