Skip to content

Conversation

@arr00
Copy link
Contributor

@arr00 arr00 commented May 6, 2024

Link T-4852

@height
Copy link

height bot commented May 6, 2024

This pull request has been linked to 1 task:


Previously linked tasks:

💡Tip: Add "Close T-4852" to the pull request title or description, to a commit message, or in a comment to mark this task as "Done" when the pull request is merged.

@arr00 arr00 force-pushed the feat/fee-on-claim branch from b142380 to b3db034 Compare May 7, 2024 16:03
@arr00 arr00 requested a review from 0xble May 7, 2024 19:57
@arr00 arr00 force-pushed the feat/fee-on-claim branch from 006bc3a to f465fb9 Compare May 8, 2024 19:04
@codecov-commenter
Copy link

codecov-commenter commented May 8, 2024

Codecov Report

Attention: Patch coverage is 81.63265% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 59.00%. Comparing base (e637f8a) to head (56e5f09).
Report is 1 commits behind head on main.

Files Patch % Lines
src/Dropper.sol 81.63% 2 Missing and 7 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   52.77%   59.00%   +6.22%     
==========================================
  Files           2        2              
  Lines          72      100      +28     
  Branches       19       28       +9     
==========================================
+ Hits           38       59      +21     
- Misses         33       34       +1     
- Partials        1        7       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@arr00 arr00 marked this pull request as ready for review May 8, 2024 19:42
Copy link
Collaborator

@0xble 0xble left a comment

Choose a reason for hiding this comment

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

Nice. Great fuzzing tests

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