Skip to content

feat(op-deployer): build contracts in docker#19088

Merged
serpixel merged 5 commits intodevelopfrom
feat/op-deployer/build-contracts-docker
Feb 6, 2026
Merged

feat(op-deployer): build contracts in docker#19088
serpixel merged 5 commits intodevelopfrom
feat/op-deployer/build-contracts-docker

Conversation

@serpixel
Copy link
Contributor

@serpixel serpixel commented Feb 4, 2026

Description

Changes the op-deployer release process to build the contracts inside the docker image.

Tests

Additional context

Metadata

@serpixel serpixel requested a review from a team as a code owner February 4, 2026 19:45
@serpixel serpixel requested a review from pauldowman February 4, 2026 19:45
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.0%. Comparing base (e28d709) to head (c9add12).
⚠️ Report is 15 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #19088     +/-   ##
==========================================
- Coverage     76.3%    76.0%   -0.4%     
==========================================
  Files          188      188             
  Lines        10943    10943             
==========================================
- Hits          8359     8326     -33     
- Misses        2438     2473     +35     
+ Partials       146      144      -2     
Flag Coverage Δ
cannon-go-tests-64 66.4% <ø> (-0.9%) ⬇️
contracts-bedrock-tests 81.7% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@serpixel serpixel force-pushed the feat/op-deployer/build-contracts-docker branch from 38e75c8 to 09de543 Compare February 4, 2026 20:11
@serpixel serpixel requested a review from a team as a code owner February 4, 2026 20:11
@serpixel serpixel force-pushed the feat/op-deployer/build-contracts-docker branch from 09de543 to 482405f Compare February 4, 2026 20:21
@serpixel serpixel force-pushed the feat/op-deployer/build-contracts-docker branch from 482405f to b44a32a Compare February 4, 2026 21:51
@serpixel serpixel enabled auto-merge February 4, 2026 22:50
Copy link
Contributor

@janjakubnanista janjakubnanista left a comment

Choose a reason for hiding this comment

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

I mean you can check how much slower the build is / how much larger the docker context is and depending on that either ignore my comment or revert to the previous version of the PR when it comes to git init

@serpixel serpixel added this pull request to the merge queue Feb 5, 2026
Merged via the queue into develop with commit 513eca1 Feb 6, 2026
209 of 211 checks passed
@serpixel serpixel deleted the feat/op-deployer/build-contracts-docker branch February 6, 2026 00:12
ajsutton added a commit that referenced this pull request Feb 6, 2026
github-merge-queue bot pushed a commit that referenced this pull request Feb 6, 2026
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.

2 participants