Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

bound (fixed) rate #128

Merged
merged 7 commits into from
Mar 5, 2024
Merged

bound (fixed) rate #128

merged 7 commits into from
Mar 5, 2024

Conversation

MathisGD
Copy link
Contributor

Fixes https://github.com/cantinasec/review-morpho/issues/18

I choose the same max rate as the one of AdaptiveCurveIRM

@MathisGD MathisGD self-assigned this Feb 28, 2024
@MathisGD MathisGD marked this pull request as ready for review February 28, 2024 09:49
Copy link
Contributor

@Rubilmax Rubilmax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How can I have missed this 🤦

test/forge/FixedRateIrmTest.sol Outdated Show resolved Hide resolved
src/fixed-rate-irm/FixedRateIrm.sol Show resolved Hide resolved
src/fixed-rate-irm/FixedRateIrm.sol Show resolved Hide resolved
test/forge/FixedRateIrmTest.sol Outdated Show resolved Hide resolved
Co-authored-by: Merlin Egalite <44097430+MerlinEgalite@users.noreply.github.com>
Signed-off-by: MathisGD <74971347+MathisGD@users.noreply.github.com>
MathisGD and others added 2 commits March 1, 2024 12:06
Signed-off-by: MathisGD <74971347+MathisGD@users.noreply.github.com>
@MathisGD MathisGD changed the base branch from feat/fixed-rate to docs/natspec-in-interface March 4, 2024 10:49
Base automatically changed from docs/natspec-in-interface to feat/fixed-rate March 5, 2024 11:17
@MathisGD MathisGD merged commit b2c9ba7 into feat/fixed-rate Mar 5, 2024
4 checks passed
@MathisGD MathisGD deleted the fix/bound-rate branch March 5, 2024 11:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants