Skip to content

Update deploy logs for optimismsepolia#1697

Merged
0xDEnYO merged 2 commits intomainfrom
deploy-polymercctp-with-testnet-to-optimismsepolia
Apr 3, 2026
Merged

Update deploy logs for optimismsepolia#1697
0xDEnYO merged 2 commits intomainfrom
deploy-polymercctp-with-testnet-to-optimismsepolia

Conversation

@mirooon
Copy link
Copy Markdown
Contributor

@mirooon mirooon commented Apr 1, 2026

Which Linear task belongs to this PR?

Why did I implement it this way?

Checklist before requesting a review

Checklist for reviewer (DO NOT DEPLOY and contracts BEFORE CHECKING THIS!!!)

  • I have checked that any arbitrary calls to external contracts are validated and or restricted
  • I have checked that any privileged calls (i.e. storage modifications) are validated and or restricted
  • I have ensured that any new contracts have had AT A MINIMUM 1 preliminary audit conducted on by <company/auditor>

@lifi-action-bot lifi-action-bot marked this pull request as draft April 1, 2026 13:44
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 1, 2026

Walkthrough

Updated the deployed address for the PolymerCCTPFacet in Optimism Sepolia deployment manifests: changed from 0x7D51DEaa90632B71087ffb164306fA192302B933 to 0xc5237Ae78f86Dc838B84fD3417fF781E77DBC6b0.

Changes

Cohort / File(s) Summary
Deployment JSON
deployments/optimismsepolia.json
Replaced PolymerCCTPFacet address value with the new address.
Diamond deployment mapping
deployments/optimismsepolia.diamond.json
Updated LiFiDiamondFacets key for PolymerCCTPFacet to the new facet address (version/name unchanged).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description follows the template structure with checklists completed where applicable, though required sections lack context: the Linear task and implementation rationale fields are empty placeholders. Fill in the 'Which Linear task belongs to this PR?' and 'Why did I implement it this way?' sections to provide full context for reviewers.
Title check ❓ Inconclusive The title 'Update deploy logs for optimismsepolia' is vague and generic, using the non-descriptive term 'deploy logs' without specifying what was actually updated. Revise the title to be more specific and descriptive, such as 'Update PolymerCCTPFacet address in optimismsepolia deployments' to clearly convey the actual change being made.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch deploy-polymercctp-with-testnet-to-optimismsepolia

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mirooon mirooon marked this pull request as ready for review April 2, 2026 11:33
@mirooon mirooon changed the title Update PolymerCCTPFacet address in optimismsepolia.json Update deploy logs for optimismsepolia Apr 2, 2026
Copy link
Copy Markdown
Contributor

@gvladika gvladika left a comment

Choose a reason for hiding this comment

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

Lgtm

@0xDEnYO 0xDEnYO merged commit 84cba7c into main Apr 3, 2026
33 of 34 checks passed
@0xDEnYO 0xDEnYO deleted the deploy-polymercctp-with-testnet-to-optimismsepolia branch April 3, 2026 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants