Skip to content

Conversation

@jjtny1
Copy link
Contributor

@jjtny1 jjtny1 commented Nov 10, 2025

[Executed]

This PR updates the Base's smart contracts using the OP Contracts Manager. Follows: https://docs.optimism.io/notices/upgrade-17

@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Nov 10, 2025

✅ Heimdall Review Status

Requirement Status More Info
Reviews 2/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 2
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 2
2
1 if commit is unverified 0
Sum 2

@jjtny1 jjtny1 marked this pull request as draft November 10, 2025 17:21
@jjtny1 jjtny1 marked this pull request as ready for review November 14, 2025 18:00
OP_COMMIT=d09c836f818c73ae139f60b717654c4e53712743
BASE_CONTRACTS_COMMIT=de19b64b38784d62606c48891d886289f2ce2e67

ABSOLUTE_PRESTATE=0x03caa1871bb9fe7f9b11217c245c16e4ded33367df5b3ccb2c6d0a847a217d1b
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jjtny1 jjtny1 requested a review from jackchuma November 14, 2025 18:07
@@ -0,0 +1,17 @@
OP_COMMIT=d09c836f818c73ae139f60b717654c4e53712743
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will make a template after this is approved

@@ -0,0 +1,17 @@
OP_COMMIT=d09c836f818c73ae139f60b717654c4e53712743
Copy link
Contributor

Choose a reason for hiding this comment

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

Confirmed this OP commit is correct and associated with op-contracts/v5.0.0-rc.2

@jjtny1 jjtny1 requested a review from jackchuma November 18, 2025 02:57
jackchuma
jackchuma previously approved these changes Nov 18, 2025
@cb-heimdall cb-heimdall dismissed jackchuma’s stale review November 18, 2025 14:14

Approved review 3477878753 from jackchuma is now dismissed due to new commit. Re-request for approval.

@jjtny1 jjtny1 changed the title feat: U17 Jovian Contracts Upgrade feat: [executed] U17 Jovian Contracts Upgrade Nov 18, 2025
@jjtny1 jjtny1 changed the title feat: [executed] U17 Jovian Contracts Upgrade feat: U17 Jovian Contracts Upgrade Nov 18, 2025
@jjtny1 jjtny1 merged commit 1e297a6 into main Nov 18, 2025
5 checks passed
@jjtny1 jjtny1 deleted the joby.thundil/u17-jovian branch November 18, 2025 16:59
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