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: add name field to IDripCheck #10732

Merged
merged 1 commit into from
Jun 4, 2024
Merged

feat: add name field to IDripCheck #10732

merged 1 commit into from
Jun 4, 2024

Conversation

smartcontracts
Copy link
Contributor

Description
Adds a new name function to IDripCheck. Makes monitoring simpler because monitoring services can now scan for drips with specific names.

@smartcontracts smartcontracts requested a review from a team as a code owner June 4, 2024 17:23
@smartcontracts smartcontracts requested a review from clabby June 4, 2024 17:23
Adds a new name function to IDripCheck. Makes monitoring simpler
because monitoring services can now scan for drips with specific
names.
@clabby clabby enabled auto-merge June 4, 2024 17:50
@clabby clabby added this pull request to the merge queue Jun 4, 2024
Copy link

codecov bot commented Jun 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.26%. Comparing base (1595335) to head (666b20f).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #10732       +/-   ##
============================================
- Coverage    54.65%   39.26%   -15.39%     
============================================
  Files           37       27       -10     
  Lines         2944     1821     -1123     
  Branches       415      415               
============================================
- Hits          1609      715      -894     
+ Misses        1303     1106      -197     
+ Partials        32        0       -32     
Flag Coverage Δ
cannon-go-tests ?
chain-mon-tests 27.14% <ø> (ø)
sdk-tests 40.27% <ø> (ø)

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

see 10 files with indirect coverage changes

Merged via the queue into develop with commit af1d939 Jun 4, 2024
67 checks passed
@clabby clabby deleted the sc/dripcheck-name branch June 4, 2024 18:12
rdovgan pushed a commit to rdovgan/optimism that referenced this pull request Jun 24, 2024
Adds a new name function to IDripCheck. Makes monitoring simpler
because monitoring services can now scan for drips with specific
names.
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