Skip to content

Conversation

@maurelian
Copy link
Contributor

Description

Adds a script for deploying the FeesDepositor. This script follows the pattern of other existing 'one off' deployment scripts (ie. DeployMips2.s.sol) as well as the tests of those scripts.

@maurelian maurelian requested a review from a team as a code owner November 8, 2025 15:20
@codecov
Copy link

codecov bot commented Nov 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.08%. Comparing base (08eb230) to head (5151ade).
⚠️ Report is 11 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #18223      +/-   ##
===========================================
+ Coverage    76.62%   82.08%   +5.46%     
===========================================
  Files          179      124      -55     
  Lines        10717     6694    -4023     
===========================================
- Hits          8212     5495    -2717     
+ Misses        2359     1199    -1160     
+ Partials       146        0     -146     
Flag Coverage Δ
cannon-go-tests-64 ?
contracts-bedrock-tests 82.08% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 55 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.

Comment on lines +87 to +88
IProxy _feesDepositorProxy,
IFeesDepositor _feesDepositorImpl,
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing natSpec for these two

Copy link
Contributor

@0xChin 0xChin left a comment

Choose a reason for hiding this comment

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

looking good!

Copy link
Contributor

@0xDiscotech 0xDiscotech left a comment

Choose a reason for hiding this comment

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

LGTM!

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.

4 participants