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

PHP 8.2 Update #34

Merged
merged 6 commits into from
Aug 22, 2024
Merged

PHP 8.2 Update #34

merged 6 commits into from
Aug 22, 2024

Conversation

rettahc
Copy link
Member

@rettahc rettahc commented Aug 22, 2024

PHP 8.2 Update & Removed Port/Spreadsheet

FBGER\dwinter added 2 commits August 22, 2024 12:00
Removed Port/Spreadsheet
Removed Port/Spreadsheet
Copy link

codecov bot commented Aug 22, 2024

Codecov Report

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

Project coverage is 76.40%. Comparing base (4e7e055) to head (f667762).

Files Patch % Lines
src/Reader/XlsxReader.php 96.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main      #34      +/-   ##
============================================
+ Coverage     75.42%   76.40%   +0.98%     
- Complexity      197      208      +11     
============================================
  Files            34       34              
  Lines           594      623      +29     
============================================
+ Hits            448      476      +28     
- Misses          146      147       +1     
Flag Coverage Δ
unittests 76.40% <96.66%> (+0.98%) ⬆️

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

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

@dhirtzbruch dhirtzbruch merged commit 446e6fb into main Aug 22, 2024
4 of 6 checks passed
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