Skip to content

Conversation

@fusmanii
Copy link
Contributor

No description provided.

Signed-off-by: Faisal Usmani <faisal.of.usmani@gmail.com>
Signed-off-by: Faisal Usmani <faisal.of.usmani@gmail.com>
@socket-security
Copy link

socket-security bot commented Sep 15, 2025

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@bmzig
Copy link
Contributor

bmzig commented Sep 15, 2025

Whenever this is ready, you'll also need to add the spoke pool proxy address to deployments.json. It's also generally nice if you add the relevant links to deployments/README.md too.

Signed-off-by: Ihor Farion <ihor@umaproject.org>
Signed-off-by: Ihor Farion <ihor@umaproject.org>
bmzig
bmzig previously approved these changes Sep 15, 2025
FILL_DEADLINE_BUFFER,
USDC[spokeChainId] ?? ZERO_ADDRESS,
// USDC[spokeChainId] ?? ZERO_ADDRESS,
"0xb88339CB7199b77E23DB6E890353E22632Ba630f", // USDC on HyperEVM
Copy link
Contributor

Choose a reason for hiding this comment

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

Before we merge this, we should get this fixed I think

"contracts": {
"SpokePool": {
"address": "0xe7cb3e167e7475dE1331Cf6E0CEb187654619E12",
"address": "0xb234cA484866c811d0e6D3318866F583781ED045",
Copy link
Contributor

Choose a reason for hiding this comment

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

This is expected, right?

Copy link
Contributor

Choose a reason for hiding this comment

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

It is, but I think we should be getting it in separately (...in advance) of this PR. We should really avoid mixing changes like this because it's easy to get mixed up. Flagged previously here: https://github.com/across-protocol/contracts/pull/1104/files/1a3bc006101a52fac9d42f53e1d2b68664d32736#r2348775844

Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved the extra changes here

grasphoper
grasphoper previously approved these changes Sep 15, 2025
Copy link
Contributor

@grasphoper grasphoper left a comment

Choose a reason for hiding this comment

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

Deployments look good to me!

@fusmanii fusmanii dismissed stale reviews from grasphoper and bmzig via 4858105 September 15, 2025 21:05
Signed-off-by: Faisal Usmani <faisal.of.usmani@gmail.com>
@fusmanii fusmanii force-pushed the faisal/hyperevm-spokepool branch from 4858105 to c5216fe Compare September 15, 2025 21:06
Signed-off-by: Faisal Usmani <faisal.of.usmani@gmail.com>
Signed-off-by: Faisal Usmani <faisal.of.usmani@gmail.com>
@fusmanii fusmanii force-pushed the faisal/hyperevm-spokepool branch from b9a3d4b to 559b975 Compare September 15, 2025 22:41
grasphoper and others added 3 commits September 15, 2025 15:56
Signed-off-by: Ihor Farion <ihor@umaproject.org>
Signed-off-by: Ihor Farion <ihor@umaproject.org>
Signed-off-by: Faisal Usmani <faisal.of.usmani@gmail.com>
@grasphoper grasphoper self-requested a review September 15, 2025 23:19
Copy link
Contributor

@grasphoper grasphoper left a comment

Choose a reason for hiding this comment

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

Deployments look good to me.

Adapter: 0x0ec70777Ac388774041dD5A1778Cdf3AF3134D2B
Spoke: 0x35E63eA3eb0fb7A3bc543C71FB66412e1F6B0E04

},
"Lens_Adapter": {
"address": "0x63AC22131eD457aeCbD63e6c4C7eeC7BBC74fF1F",
"address": "0x5e0B7e20a77BDf11812837D30F1326068Bcf24Cf",
Copy link
Member

Choose a reason for hiding this comment

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

Why are so many contracts unrelated to 999 changing in this file?

@fusmanii fusmanii merged commit 26564c1 into master Sep 16, 2025
10 checks passed
@fusmanii fusmanii deleted the faisal/hyperevm-spokepool branch September 16, 2025 01:22
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.

6 participants