fix: update sepolia owner #335
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the Arbitrum Sepolia testnet configuration to use a new owner/deployer address and adds documentation for new features in the v6.2.0 upgrade.
- Updates the Arbitrum Sepolia configuration (chain ID 421614) to use address
0x87bdca5F82cD5B138cD12a724d6c0eC7e3114dA6for deployer, owner, and richman roles - Adds documentation for cross-framework TEE compatibility and single-transaction deposit/match order features to the v6.2.0 upgrade notes
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| config/config.json | Updates deployer, owner, and richman addresses for Arbitrum Sepolia testnet configuration from 0x9990cfb1Feb7f47297F54bef4d4EbeDf6c5463a3 to 0x87bdca5F82cD5B138cD12a724d6c0eC7e3114dA6 |
| scripts/upgrades/v6.2.0.md | Adds a "New Features" section documenting cross-framework TEE compatibility (#325) and atomic deposit/match orders functionality (#316) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ### New Features | ||
|
|
||
| - **Cross-framework TEE compatibility (#325)**: Ignore TEE framework bits (Scone, Gramine, TDX) in dataset tags during order matching. This allows existing dataset orders with SGX tags (e.g., Scone `0b0011`) to be consumed on TDX workerpools (`0b1001`), enabling cross-framework compatibility. | ||
|
|
||
| - **Deposit and match orders in a single transaction (#316)**: Users can now call `RLC.approveAndCall()` to deposit tokens and execute `matchOrders` atomically in a single transaction. This improves UX by reducing the number of transactions required to create a deal. |
There was a problem hiding this comment.
The addition of "New Features" section appears to be unrelated to the PR title "Fix: update sepolia owner". This documentation change describes cross-framework TEE compatibility and deposit/match orders functionality, which are separate features from updating the owner address. Consider moving these documentation changes to a separate PR focused on documenting these features, or update the PR title to reflect that this PR includes both the owner update and feature documentation.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #335 +/- ##
=======================================
Coverage 96.39% 96.39%
=======================================
Files 33 33
Lines 1137 1137
Branches 228 228
=======================================
Hits 1096 1096
Misses 41 41 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
No description provided.