Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
simplify interaction with multisig wallets in deployment scripts #1859
simplify interaction with multisig wallets in deployment scripts #1859
Changes from 21 commits
a82aa8b
eeb5487
eb3b549
286c14d
5edd532
1a77382
443f373
a084678
5e6d32b
35293e0
54b7aa1
80db3c0
439b82d
77c5525
9c7606a
c1b7b6a
75842cd
10874e9
d705835
3085a65
4ac6ca7
017f487
391ce1c
d5d8c3c
c61a520
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
For my own understanding, Are these run files used by something in the repo like CI or tests?
If we do need to be committing them to the repo, what are we using them for? If not should we add the
/contracts/broadcast
directory to the .gitignore and remove ones that currently reside in the repo?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 idea is to only commit the files for public deployments to the repo. It's to make life easy for anyone who wants to interact with the deployed contracts.