ImperialCollegeLondon / pycsvy

Python reader/writer for CSV files with YAML header information
BSD 3-Clause "New" or "Revised" License
7 stars 8 forks source link

Add read_into_dict function for simpler CSVY parsing #132

Open Kaos599 opened 3 weeks ago

Kaos599 commented 3 weeks ago

Closes #91

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 92.85714% with 2 lines in your changes missing coverage. Please review.

Project coverage is 93.38%. Comparing base (05e324e) to head (7ea22ba). Report is 120 commits behind head on develop.

Files with missing lines Patch % Lines
csvy/readers.py 92.85% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #132 +/- ## =========================================== - Coverage 93.58% 93.38% -0.21% =========================================== Files 3 4 +1 Lines 156 272 +116 =========================================== + Hits 146 254 +108 - Misses 10 18 +8 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Kaos599 commented 3 weeks ago

@dalonsoa @AdrianDAlessandro

I think everything is fine except this error by Codecov.

csvy/readers.py#L307-L308
Added lines #L307 - L308 were not covered by tests

Refering to these lines.