Skip to content

delete code blocks containing TextFileReader#23

Closed
ludwiglierhammer wants to merge 115 commits intoglamod:mainfrom
ludwiglierhammer:no_textfilereader
Closed

delete code blocks containing TextFileReader#23
ludwiglierhammer wants to merge 115 commits intoglamod:mainfrom
ludwiglierhammer:no_textfilereader

Conversation

@ludwiglierhammer
Copy link
Collaborator

@codecov
Copy link

codecov bot commented Jun 26, 2024

Codecov Report

Attention: Patch coverage is 91.66667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 82.24%. Comparing base (c70c918) to head (f3de2a4).
Report is 22 commits behind head on main.

Files with missing lines Patch % Lines
cdm_reader_mapper/cdm_mapper/mapper.py 91.30% 2 Missing ⚠️
cdm_reader_mapper/mdf_reader/utils/auxiliary.py 88.23% 2 Missing ⚠️
cdm_reader_mapper/cdm_mapper/table_writer.py 80.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #23      +/-   ##
==========================================
- Coverage   82.52%   82.24%   -0.29%     
==========================================
  Files          69       68       -1     
  Lines        2638     2399     -239     
==========================================
- Hits         2177     1973     -204     
+ Misses        461      426      -35     
Flag Coverage Δ
unittests 82.24% <91.66%> (-0.29%) ⬇️

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.

@github-actions
Copy link

github-actions bot commented Jan 6, 2025

Warning

This Pull Request modifies GitHub Workflows and is coming from a fork.
It is very important for the reviewer to ensure that the workflow changes are appropriate.

@github-actions
Copy link

github-actions bot commented Jan 6, 2025

Warning

This Pull Request modifies GitHub Workflows and is coming from a fork.
It is very important for the reviewer to ensure that the workflow changes are appropriate.

@ludwiglierhammer
Copy link
Collaborator Author

This PR is out of date. We'll close this and continue in #348.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments