-
Notifications
You must be signed in to change notification settings - Fork 17
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
build: devnet-contract-deploy #222
base: main
Are you sure you want to change the base?
Conversation
|
||
- name: Commit and push | ||
run: | | ||
git add . |
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.
Optional, but might be good to target the directory where we expect changes. Helps to document it a bit better.
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.
We may need to generate binding, which is not in hardhat directory
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.
I think @bgins means: git add pkg/web3/bindings
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.
Have you identified which files are going to be added here?
Can you list them?
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.
Folders:
* pkg/web3/bindings
* hardhat/deployments
* hardhat/.openzeppelin
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 you have your list for git add
Co-authored-by: Brian Ginsburg <[email protected]>
Co-authored-by: Brian Ginsburg <[email protected]>
bbde6da
to
03d98a7
Compare
Co-authored-by: Brian Ginsburg <[email protected]>
03d98a7
to
ee2521e
Compare
@alvin-reyes this PR was addressing the 2nd step in our current 2 step process when deploying changes to the contracts. Let's put a pin on this PR until you've defined and documented our processes, and then we can decide whether we use this PR as base or close this and open a new one for the automations for the CI pipeline. @arsen3d I'm moving this PR back to draft. |
Added to engineering board assigned to me |
Review Type Requested (choose one):
Summary
Provides a way to migrate contracts on devnet.