-
Notifications
You must be signed in to change notification settings - Fork 32
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
[DO NOT MERGE] test maintenance #2841
Conversation
Important Review skippedIgnore keyword(s) in the title. Ignored keywords (2)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
- Replaced 'doge-chain-pause' entry with 'base-chain-pause'
- Added 'arbitrum-chain-pause' entry
- Added 'optimism-chain-pause' entry
- Added 'blast-chain-pause' entry
- Updated pause details: chain IDs, pause times, messages
File: packages/synapse-interface/public/pauses/v1/paused-chains.json
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
Deploying sanguine-fe with Cloudflare Pages
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2841 +/- ##
====================================================
+ Coverage 25.91575% 90.47619% +64.56044%
====================================================
Files 751 54 -697
Lines 53890 1008 -52882
Branches 80 80
====================================================
- Hits 13966 912 -13054
+ Misses 38480 93 -38387
+ Partials 1444 3 -1441
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Extended end times for maintenance banners for Optimism and Blast chains
- Extended pause chains for Optimism and Blast chains
File: packages/synapse-interface/public/pauses/v1/paused-chains.json
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Added two new paused bridge modules in
packages/synapse-interface/public/pauses/v1/paused-bridge-modules.json
- Updated start and end times for chain pauses in
packages/synapse-interface/public/pauses/v1/paused-chains.json
- Modified disable flags for banners, warnings, and countdowns in
packages/synapse-interface/public/pauses/v1/paused-chains.json
2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated end times for paused chains: Arbitrum, Optimism, Blast
- Ensured accurate maintenance periods in
packages/synapse-interface/public/pauses/v1/paused-chains.json
No major changes found since last review.
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated end times for paused chains: Arbitrum, Optimism, Blast
- Extended maintenance periods in
packages/synapse-interface/public/pauses/v1/paused-chains.json
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Extended end times for paused chains: Arbitrum, Optimism, Blast
- Updated maintenance periods in
packages/synapse-interface/public/pauses/v1/paused-chains.json
No major changes found since last review.
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated
pausedFromChains
andpausedToChains
forarbitrum-chain-pause
from[42161]
to[111]
inpackages/synapse-interface/public/pauses/v1/paused-chains.json
- Ensure the new chain ID
[111]
is correct and intended for Arbitrum configuration
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated
pausedFromChains
forarbitrum-chain-pause
from[111]
to[112]
inpackages/synapse-interface/public/pauses/v1/paused-chains.json
- Ensure the new chain ID
[112]
is correct and intended for Arbitrum configuration
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated
chainId
from 42161 to 111 inpackages/synapse-interface/public/pauses/v1/paused-bridge-modules.json
- Added new entry for
chainId
10 withbridgeModuleName
'SynapseRFQ' inpackages/synapse-interface/public/pauses/v1/paused-bridge-modules.json
- Modified
pausedFromChains
andpausedToChains
for 'base-chain-pause' inpackages/synapse-interface/public/pauses/v1/paused-chains.json
- Modified
pausedFromChains
andpausedToChains
for 'arbitrum-chain-pause' inpackages/synapse-interface/public/pauses/v1/paused-chains.json
2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated
pausedFromChains
andpausedToChains
for 'base-chain-pause' from 111 to 8453 inpackages/synapse-interface/public/pauses/v1/paused-chains.json
- Changed
pausedFromChains
for 'arbitrum-chain-pause' from 420 to 42161 inpackages/synapse-interface/public/pauses/v1/paused-chains.json
- Set
pausedToChains
for 'arbitrum-chain-pause' to an empty array inpackages/synapse-interface/public/pauses/v1/paused-chains.json
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated start and end times for Arbitrum, Optimism, and Blast chain pauses in
packages/synapse-interface/public/pauses/v1/paused-chains.json
- Set some end times to
null
for indefinite pauses
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Updated
pausedFromChains
forarbitrum-chain-pause
from[42161]
to[111]
inpackages/synapse-interface/public/pauses/v1/paused-chains.json
- No other changes to the configuration
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
null
f693a38: synapse-interface preview link
1906118: synapse-interface preview link
90f1fd1: synapse-interface preview link
0631510: synapse-interface preview link
905e519: synapse-interface preview link
9e60658: synapse-interface preview link
bbbeb4c: synapse-interface preview link
078ff8d: synapse-interface preview link
97c8a7b: synapse-interface preview link
9271335: synapse-interface preview link
948ea1e: synapse-interface preview link
af43926: synapse-interface preview link