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

Add coverage for code in doc/source/make_tables.py #5921

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

user27182
Copy link
Contributor

Add a new doc test to be executed during the Build Documentation work flow to ensure there is coverage for some of the documentation source code. Specifically, tests are added for the make_tables.py module .

This does two things:

@user27182
Copy link
Contributor Author

pre-commit.ci autofix

@github-actions github-actions bot added the maintenance Low-impact maintenance activity label Apr 12, 2024
Copy link

codecov bot commented Apr 12, 2024

Codecov Report

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

Project coverage is 96.23%. Comparing base (f9d2d86) to head (e953729).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5921      +/-   ##
==========================================
- Coverage   96.83%   96.23%   -0.61%     
==========================================
  Files         139      139              
  Lines       24039    24498     +459     
==========================================
+ Hits        23279    23575     +296     
- Misses        760      923     +163     

@user27182 user27182 marked this pull request as draft April 12, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Low-impact maintenance activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant