Skip to content
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

Moonwell EURC #1189

Merged
merged 20 commits into from
Oct 28, 2024
Merged

Moonwell EURC #1189

merged 20 commits into from
Oct 28, 2024

Conversation

jalextowle
Copy link
Contributor

Description

This PR adds testing for the Moonwell EURC yield source. Unlike most of our other yield sources, this yield source has a base decimals of 6 and vault share decimals of 18. Furthermore, the vault share price is quite low at around ~1e6. With this in mind, some of the tolerances are significantly larger than what we'd expect to see with other yield sources.

@jalextowle jalextowle changed the title Moonwell EURC WIP: Moonwell EURC Oct 11, 2024
@jalextowle jalextowle changed the title WIP: Moonwell EURC Moonwell EURC Oct 12, 2024
@jalextowle jalextowle force-pushed the jalextowle/integration/moonwell-eurc branch from 786e0d8 to e00a140 Compare October 15, 2024 21:19
@jalextowle jalextowle force-pushed the jalextowle/integration/moonwell-eurc branch from 9cc7a79 to 8352d17 Compare October 16, 2024 21:07
@jalextowle jalextowle changed the base branch from main to jalextowle/integration/snars October 16, 2024 21:08
Copy link
Contributor

@mcclurejt mcclurejt left a comment

Choose a reason for hiding this comment

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

🤘

@jalextowle jalextowle force-pushed the jalextowle/integration/moonwell-eurc branch from 01e6363 to 3212480 Compare October 18, 2024 20:01
@jalextowle jalextowle changed the base branch from jalextowle/integration/snars to main October 18, 2024 20:02
@jalextowle jalextowle mentioned this pull request Oct 18, 2024
@jalextowle jalextowle added this pull request to the merge queue Oct 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 18, 2024
@jalextowle jalextowle added this pull request to the merge queue Oct 18, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 18, 2024
@coveralls
Copy link
Collaborator

coveralls commented Oct 22, 2024

Pull Request Test Coverage Report for Build 11563543464

Details

  • 5 of 5 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 88.889%

Totals Coverage Status
Change from base Build 11507869923: 0.05%
Covered Lines: 2712
Relevant Lines: 3051

💛 - Coveralls

@jalextowle jalextowle force-pushed the jalextowle/integration/moonwell-eurc branch from ca44549 to b8d55ed Compare October 28, 2024 22:16
@jalextowle jalextowle enabled auto-merge October 28, 2024 22:17
@jalextowle jalextowle added this pull request to the merge queue Oct 28, 2024
Merged via the queue into main with commit 9c6460b Oct 28, 2024
34 of 35 checks passed
@jalextowle jalextowle deleted the jalextowle/integration/moonwell-eurc branch October 28, 2024 23:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants