-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
contracts-bedrock: slight cleanup + docs #6751
Conversation
|
Makes it easier to do deployments with a better abstraction. The first step of a deployment is always to do the deployment, then run `sync()` to parse the forge deploy artifact into a hardhat style artifact. This commit improves the UX of `sync()`
0bc6acb
to
44c8709
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## develop #6751 +/- ##
===========================================
- Coverage 44.75% 42.53% -2.23%
===========================================
Files 375 336 -39
Lines 29474 28277 -1197
Branches 1692 1541 -151
===========================================
- Hits 13192 12028 -1164
Misses 14840 14840
+ Partials 1442 1409 -33
Flags with carried forward coverage won't be shown. Click here to find out more. |
This PR has been added to the merge queue, and will be merged soon. |
ty for fix @refcell <3 |
This PR is next in line to be merged, and will be merged as soon as checks pass. |
1 similar comment
This PR is next in line to be merged, and will be merged as soon as checks pass. |
Description
Makes it easier to do deployments with a better abstraction. The first step of a deployment is always to do the deployment, then run
sync()
to parse the forge deploy artifact into a hardhat style artifact. This commit improves the UX ofsync()