Skip to content

resolv throw#18512

Merged
waynebruce0x merged 1 commit intomainfrom
resolv-throw
Mar 24, 2026
Merged

resolv throw#18512
waynebruce0x merged 1 commit intomainfrom
resolv-throw

Conversation

@waynebruce0x
Copy link
Collaborator

@waynebruce0x waynebruce0x commented Mar 24, 2026

Summary by CodeRabbit

  • Chores
    • The Resolve Ethereum adapter has been temporarily disabled and will throw an error indicating it requires maintenance. Staking functionality remains operational.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 92143608-6d08-4d86-9875-ef1142691adf

📥 Commits

Reviewing files that changed from the base of the PR and between 244453e and f94d4c7.

📒 Files selected for processing (1)
  • projects/resolv/index.js

📝 Walkthrough

Walkthrough

The tvl function for the Ethereum chain in the Resolv adapter is replaced, transitioning from aggregating token total supplies via multiCall to immediately throwing a runtime error.

Changes

Cohort / File(s) Summary
Resolv Adapter TVL Update
projects/resolv/index.js
The Ethereum chain's tvl method is modified to throw an error instead of performing token supply aggregation via multiCall.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 The Resolv adapter needs some love,
Its tvl function won't fit like a glove,
An error it throws with a simple "fix me,"
A marker for change that soon we shall see!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch resolv-throw

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@waynebruce0x waynebruce0x merged commit 80154e8 into main Mar 24, 2026
1 of 2 checks passed
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.

1 participant