Closed horsefacts closed 1 year ago
Coverage after merging horsefacts/forge-install-docker into main will be
98.79% |
---|
File | Stmts | Branches | Funcs | Lines | Uncovered Lines |
---|---|---|---|---|---|
src | |||||
Bundler.sol | 100% | 100% | 100% | 100% | |
FnameResolver.sol | 100% | 100% | 100% | 100% | |
IdRegistry.sol | 100% | 100% | 100% | 100% | |
KeyRegistry.sol | 100% | 100% | 100% | 100% | |
StorageRegistry.sol | 100% | 100% | 100% | 100% | |
src/lib | |||||
Signatures.sol | 100% | 100% | 100% | 100% | |
TransferHelper.sol | 0% | 0% | 0% | 0% | 12, 17, 20, 20, 20 |
TrustedCaller.sol | 100% | 100% | 100% | 100% |
Motivation
The
contracts-deployer
docker-compose service fails if the user hasn't installed forge dependencies in their local repository.Change Summary
Add a
forge install
step to thecontracts-deployer
startup script.Merge Checklist
Choose all relevant options below by adding an
x
now or at any time before submitting for reviewAdditional Context
If this is a relatively large or complex change, provide more details here that will help reviewers.
PR-Codex overview
This PR focuses on deploying a contract and disabling impersonation.
Detailed summary
forge install
command for deploymentforge script
command for executing a scriptcast rpc
command for enabling and disabling impersonation