-
Notifications
You must be signed in to change notification settings - Fork 5
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
remove parameter and cleanup names from calc_fees functions #656
Conversation
Hyperdrive Gas Benchmark
This comment was automatically generated by workflow using github-action-benchmark. |
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.
Nice work overall - the notes in the PR description are great! Some nits to address. I am going to wait to approve until @jalextowle can review and the issues have been resolved.
test/integrations/hyperdrive/NegativeInterestShortFeeTest.t.sol
Outdated
Show resolved
Hide resolved
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.
Nice cleanup! These changes are great. Let me know once the nits are addressed, and I'll take another look.
test/integrations/hyperdrive/NegativeInterestShortFeeTest.t.sol
Outdated
Show resolved
Hide resolved
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.
LGTM
5ce5778
to
7971e5e
Compare
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.
Looks good now. i'll hold off until alex approves
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.
LGTM
governanceFlatFee
return varaiabletotalCurveFee
->curveFee
,totalFlatFee
->flatFee
and_amount
to_shareAmount
and_bondAmount
as appropriate (a few tests still usetotalCurveFee = curveFee * amount
)