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

fix Epub empty array with getChapters func #91

Merged
merged 1 commit into from
Jul 16, 2024

Conversation

basitcodeenv
Copy link
Contributor

I upgrade php-ebook package to latest version and It still was not working because I forgot to remove one line in my Previous PR which I have done in this PR.

Copy link

codecov bot commented Jul 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.54%. Comparing base (4eec489) to head (78edbce).
Report is 8 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #91      +/-   ##
============================================
- Coverage     90.55%   90.54%   -0.01%     
  Complexity      826      826              
============================================
  Files            32       32              
  Lines          2540     2539       -1     
============================================
- Hits           2300     2299       -1     
  Misses          240      240              

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

@ewilan-riviere
Copy link
Contributor

Okay, no problem! I merge this one!

@ewilan-riviere ewilan-riviere merged commit 0b021a6 into kiwilan:main Jul 16, 2024
9 checks passed
ewilan-riviere added a commit that referenced this pull request Jul 16, 2024
Hotfix PR #91 for issue #71 by @basitcodeenv, thanks!
@ewilan-riviere ewilan-riviere mentioned this pull request Jul 16, 2024
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