Closed jalextowle closed 1 month ago
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contracts/src/factory/HyperdriveFactory.sol | 13 | 17 | 76.47% | ||
contracts/src/factory/HyperdriveRegistry.sol | 83 | 92 | 90.22% | ||
<!-- | Total: | 96 | 109 | 88.07% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contracts/src/factory/HyperdriveRegistry.sol | 1 | 89.36% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 9334613106: | -0.3% |
Covered Lines: | 1909 |
Relevant Lines: | 2052 |
Benchmark suite | Current: dc558a619efe3a64f48290d32a3ac83bde513294 | Previous: d47b4e99631301d74daf01fc47681b3ce50beac9 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33827 gas |
33827 gas |
0% |
π° |
addLiquidity: avg |
155096 gas |
155515 gas |
-0.2694% |
β |
addLiquidity: max |
429092 gas |
428173 gas |
0.2146% |
π¨ |
checkpoint: min |
40292 gas |
40292 gas |
0% |
π° |
checkpoint: avg |
142270 gas |
142272 gas |
-0.0014% |
β |
checkpoint: max |
253424 gas |
253424 gas |
0% |
π° |
closeLong: min |
31361 gas |
31361 gas |
0% |
π° |
closeLong: avg |
136101 gas |
134707 gas |
1.0348% |
π¨ |
closeLong: max |
2625796 gas |
2625796 gas |
0% |
π° |
closeShort: min |
31349 gas |
31349 gas |
0% |
π° |
closeShort: avg |
132110 gas |
131060 gas |
0.8012% |
π¨ |
closeShort: max |
309547 gas |
262384 gas |
17.9748% |
π¨ |
initialize: min |
31371 gas |
31371 gas |
0% |
π° |
initialize: avg |
331019 gas |
330116 gas |
0.2735% |
π¨ |
initialize: max |
397010 gas |
396015 gas |
0.2513% |
π¨ |
openLong: min |
33370 gas |
33370 gas |
0% |
π° |
openLong: avg |
174071 gas |
172914 gas |
0.6691% |
π¨ |
openLong: max |
306958 gas |
306039 gas |
0.3003% |
π¨ |
openShort: min |
33936 gas |
33936 gas |
0% |
π° |
openShort: avg |
168677 gas |
167433 gas |
0.7430% |
π¨ |
openShort: max |
415814 gas |
414787 gas |
0.2476% |
π¨ |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
π° |
redeemWithdrawalShares: avg |
75041 gas |
74277 gas |
1.0286% |
π¨ |
redeemWithdrawalShares: max |
210204 gas |
209214 gas |
0.4732% |
π¨ |
removeLiquidity: min |
31301 gas |
31301 gas |
0% |
π° |
removeLiquidity: avg |
214444 gas |
208376 gas |
2.9120% |
π¨ |
removeLiquidity: max |
403959 gas |
403026 gas |
0.2315% |
π¨ |
This comment was automatically generated by workflow using github-action-benchmark.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contracts/src/factory/HyperdriveFactory.sol | 13 | 17 | 76.47% | ||
contracts/src/factory/HyperdriveRegistry.sol | 83 | 92 | 90.22% | ||
<!-- | Total: | 96 | 109 | 88.07% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contracts/src/factory/HyperdriveRegistry.sol | 1 | 89.36% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 9374786147: | -0.4% |
Covered Lines: | 1908 |
Relevant Lines: | 2052 |
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
contracts/src/factory/HyperdriveFactory.sol | 13 | 17 | 76.47% | ||
contracts/src/factory/HyperdriveRegistry.sol | 83 | 92 | 90.22% | ||
<!-- | Total: | 96 | 109 | 88.07% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
contracts/src/factory/HyperdriveRegistry.sol | 1 | 89.36% | ||
<!-- | Total: | 1 | --> |
Totals | |
---|---|
Change from base Build 9392859661: | -0.4% |
Covered Lines: | 1908 |
Relevant Lines: | 2052 |
Description
Updates the schema for the registry so that it is more helpful for programmatic users. This PR also updates some inconsistencies in the
HyperdriveFactory
and interfaces throughout the codebase.Review Checklists
Please check each item before approving the pull request. While going through the checklist, it is recommended to leave comments on items that are referenced in the checklist to make sure that they are reviewed. If there are multiple reviewers, copy the checklists into sections titled
## [Reviewer Name]
. If the PR doesn't touch Solidity, the corresponding checklist can be removed.[[Reviewer Name]]
approve
calls useforceApprove
?transfer
calls usesafeTransfer
?transferFrom
calls usemsg.sender
as thefrom
address?call
,delegatecall
,staticcall
,transfer
,send
)success
boolean checked to handle failed calls?delegatecall
, are there strict access controls on the addresses that can be called? It shouldn't be possible todelegatecall
arbitrary addresses, so the list of possible targets should either be immutable or tightly controlled by an admin.nonReentrant
?payable
functions restricted to avoid stuck ether?Safe
functions are altered, are potential underflows and overflows caught so that a failure flag can be thrown?