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.

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

chore: reproduce deployment broadcast files from on-chain data #204

Merged
merged 4 commits into from
Aug 13, 2024

Conversation

bowd
Copy link
Contributor

@bowd bowd commented Aug 8, 2024

Description

When I deployed and executed the governance proposal for Mento Governance, I made the immense mistake of not committing the broadcast files to Github. A few days later my macbook went belly up and I lost all the data. Sigh.

This PR uses an archive node (quicknode) and some manual data to reconstruct the Foundry broadcast files. This is useful to have in conjunction with the rest of our tooling which depends on broadcast files. It would have been very useful a few months ago because it makes it really easy to verify contracts on celoscan, I used it to verify a few remaining proxies.

The script to generate this is a one-off script (famous last words), but I thought it made sense to put it in there in case we ever need something similar, but it shouldn't really be reviewed thoroughly.

Other changes

N/A

Tested

Verified broadcast script manually and tried it out with the cleoscan verification script.

Related issues

N/A

Backwards compatibility

Precisely

Documentation

N/A

bin/reproduce-mainnet-gov.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@baroooo baroooo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added slight modifications to reduce the differences between the foundry generated and script generated broadcast files. Not a blocker for this, feel free to re-gen broadcast files if you see a need.

bin/reproduce-mainnet-gov.ts Show resolved Hide resolved
bin/reproduce-mainnet-gov.ts Outdated Show resolved Hide resolved
bin/reproduce-mainnet-gov.ts Show resolved Hide resolved
bin/reproduce-mainnet-gov.ts Outdated Show resolved Hide resolved
@bowd bowd merged commit 9a4fecf into main Aug 13, 2024
1 check passed
@bowd bowd deleted the fix/reconstruct-mainnet-gov-broadcasts branch August 13, 2024 09:47
bowd added a commit that referenced this pull request Aug 15, 2024
* chore: reproduce deployment broadcast files from on-chain data

* Update bin/reproduce-mainnet-gov.ts

Co-authored-by: Ryan Noble <[email protected]>

* Apply suggestions from code review

Co-authored-by: baroooo <[email protected]>

* feat: regenerate json files after Baran's suggestions

---------

Co-authored-by: Ryan Noble <[email protected]>
Co-authored-by: baroooo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants