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

Handle libretexts.org glossary rewriting #66

Merged
merged 3 commits into from
Nov 13, 2024

Add sample path and URL for glossary pages

477304f
Select commit
Loading
Failed to load commit list.
Merged

Handle libretexts.org glossary rewriting #66

Add sample path and URL for glossary pages
477304f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Nov 13, 2024 in 0s

44.43% (-0.66%) compared to f0e6cfe

View this Pull Request on Codecov

44.43% (-0.66%) compared to f0e6cfe

Details

Codecov Report

Attention: Patch coverage is 22.22222% with 21 lines in your changes missing coverage. Please review.

Project coverage is 44.43%. Comparing base (f0e6cfe) to head (477304f).

Files with missing lines Patch % Lines
scraper/src/mindtouch2zim/libretexts/glossary.py 23.80% 16 Missing ⚠️
scraper/src/mindtouch2zim/processor.py 16.66% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #66      +/-   ##
==========================================
- Coverage   45.08%   44.43%   -0.66%     
==========================================
  Files          13       14       +1     
  Lines         845      871      +26     
  Branches      111      116       +5     
==========================================
+ Hits          381      387       +6     
- Misses        451      471      +20     
  Partials       13       13              

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