Skip to content

Update Vault Registry#23

Open
jdomingos wants to merge 1 commit into
mainfrom
update-vault-registry
Open

Update Vault Registry#23
jdomingos wants to merge 1 commit into
mainfrom
update-vault-registry

Conversation

@jdomingos
Copy link
Copy Markdown
Contributor

No description provided.

@jdomingos jdomingos requested review from Philippoes and ajag408 May 29, 2026 10:34
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 29, 2026

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • src/validators/evm/erc4626/vault-registry.json
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9baae096-8ae1-4815-a5dd-29dde0dc38c3

📥 Commits

Reviewing files that changed from the base of the PR and between 9b8f7c3 and 2e90510.

📒 Files selected for processing (1)
  • src/validators/evm/erc4626/vault-registry.json

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-vault-registry

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

Copy link
Copy Markdown

@raiseerco raiseerco left a comment

Choose a reason for hiding this comment

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

binance-wbnb-* was flipped to isWethVault: false, which disables wrap-unwrap validation for native BNB flows, looks like these should stay true

Copy link
Copy Markdown

@raiseerco raiseerco left a comment

Choose a reason for hiding this comment

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

only one

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.

2 participants