Skip to content

Conversation

@vjik
Copy link
Member

@vjik vjik commented Feb 3, 2025

Q A
Is bugfix? ✔️
New feature?
Breaks BC?

@codecov
Copy link

codecov bot commented Feb 3, 2025

Codecov Report

Attention: Patch coverage is 92.85714% with 1 line in your changes missing coverage. Please review.

Project coverage is 77.47%. Comparing base (b5c1568) to head (9e6b3e1).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/Exception/ErrorException.php 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #139      +/-   ##
============================================
- Coverage     77.81%   77.47%   -0.35%     
  Complexity      184      184              
============================================
  Files            14       14              
  Lines           586      586              
============================================
- Hits            456      454       -2     
- Misses          130      132       +2     

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

@vjik vjik requested a review from a team February 3, 2025 12:17
@vjik vjik added the status:code review The pull request needs review. label Feb 3, 2025
@samdark samdark merged commit 91bab7a into master Feb 4, 2025
22 of 23 checks passed
@samdark samdark deleted the php84 branch February 4, 2025 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants