Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: introduce op-dripper #13788

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

feat: introduce op-dripper #13788

wants to merge 1 commit into from

Conversation

smartcontracts
Copy link
Contributor

Description

Introduces op-dripper as a replacement for Gelato automation of Drippie. Gelato no longer supports querying balances onchain which makes self-sustaining automation difficult. This PR removes all references to Gelato and introduces the new service.

Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.68%. Comparing base (3cc36be) to head (83983ce).
Report is 9 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #13788      +/-   ##
===========================================
- Coverage    47.03%   45.68%   -1.36%     
===========================================
  Files          957      899      -58     
  Lines        80029    75249    -4780     
  Branches       758      758              
===========================================
- Hits         37642    34377    -3265     
+ Misses       39529    38188    -1341     
+ Partials      2858     2684     -174     
Flag Coverage Δ
cannon-go-tests-32 ?
cannon-go-tests-64 ?
contracts-bedrock-tests 88.97% <ø> (-0.09%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 72 files with indirect coverage changes

@smartcontracts smartcontracts force-pushed the sc/op-dripper branch 2 times, most recently from b75bce8 to 1588bbb Compare January 15, 2025 17:52
@tynes
Copy link
Contributor

tynes commented Jan 15, 2025

Does this need to live in the monorepo?

@smartcontracts
Copy link
Contributor Author

Does this need to live in the monorepo?

@mslipper suggested that it live in the monorepo. Balance management is a relatively core part of the system IMO, so I think it seems reasonable to have it live here.

@ControlCplusControlV
Copy link
Contributor

Where else would it go besides the monorepo? Monitorism isn't quite the right place imo because its critical to production in keeping things funded

@smartcontracts
Copy link
Contributor Author

As a side note, it would be nice if we moved the Drippie configuration and management tooling into this package to cut out stuff from the contracts package.

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.

3 participants