feat: add USDnr (Nerona USD) warp route between Ethereum and Fluent#1476
feat: add USDnr (Nerona USD) warp route between Ethereum and Fluent#1476Khrafts wants to merge 1 commit intohyperlane-xyz:mainfrom
Conversation
Uses the EvmM0Portal standard.
📝 WalkthroughWalkthroughA changeset entry and warp route configuration file were added to support a new USDnr token bridge between Ethereum and Fluent chains. The configuration specifies token metadata, decimal precision, and cross-chain connection mappings using the EvmM0Portal standard. Changes
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
deployments/warp_routes/warpRouteConfigs.yaml (1)
11104-11127: Drop this manualwarpRouteConfigs.yamledit from the PR.This index file is CI-generated after merge, so hand-editing it here is unnecessary and can cause avoidable conflicts with other route additions.
Based on learnings: In the hyperlane-xyz/hyperlane-registry repo,
deployments/warp_routes/warpRouteConfigs.yamlis automatically updated by a CI job that runs after a PR is merged, so manual edits are not required.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@deployments/warp_routes/warpRouteConfigs.yaml` around lines 11104 - 11127, This PR should remove the manual edits to the CI-generated warpRouteConfigs.yaml: delete the added USDnr/usdnr token entries (the block defining USDnr/usdnr with tokens and both ethereum and fluent entries) and any other hand-modifications to deployments/warp_routes/warpRouteConfigs.yaml so the file remains untouched in the PR; the CI job will regenerate and merge the canonical changes after PR merge.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.changeset/quick-terms-begin.md:
- Line 5: Update the release note text in .changeset/quick-terms-begin.md by
correcting the chain name capitalization from "FLuent" to "Fluent" in the line
containing "Add USDnr (Nerona USD) warp route between Ethereum and FLuent using
EvmM0Portal standard" so the sentence reads with "Fluent" for consistent,
searchable release notes.
---
Nitpick comments:
In `@deployments/warp_routes/warpRouteConfigs.yaml`:
- Around line 11104-11127: This PR should remove the manual edits to the
CI-generated warpRouteConfigs.yaml: delete the added USDnr/usdnr token entries
(the block defining USDnr/usdnr with tokens and both ethereum and fluent
entries) and any other hand-modifications to
deployments/warp_routes/warpRouteConfigs.yaml so the file remains untouched in
the PR; the CI job will regenerate and merge the canonical changes after PR
merge.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: a14aa9a7-08c9-4f21-8c93-d584ead125e4
⛔ Files ignored due to path filters (1)
deployments/warp_routes/USDnr/logo.svgis excluded by!**/*.svg
📒 Files selected for processing (3)
.changeset/quick-terms-begin.mddeployments/warp_routes/USDnr/usdnr-config.yamldeployments/warp_routes/warpRouteConfigs.yaml
| '@hyperlane-xyz/registry': patch | ||
| --- | ||
|
|
||
| Add USDnr (Nerona USD) warp route between Ethereum and FLuent using EvmM0Portal standard |
There was a problem hiding this comment.
Fix chain name capitalization in release note text.
On Line 5, FLuent should be Fluent to keep release notes polished and searchable.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.changeset/quick-terms-begin.md at line 5, Update the release note text in
.changeset/quick-terms-begin.md by correcting the chain name capitalization from
"FLuent" to "Fluent" in the line containing "Add USDnr (Nerona USD) warp route
between Ethereum and FLuent using EvmM0Portal standard" so the sentence reads
with "Fluent" for consistent, searchable release notes.
Uses the EvmM0Portal standard.
Description
Example: Add USDnr warp route on Ethereum <-> Fluent
Backward compatibility
Yes
Testing