search

mini-pw / PvSTATEM

R package for automated analysis of serological data
https://mini-pw.github.io/PvSTATEM/
Other
2 stars 2 forks source link

splitting class definitions #55

Closed nizwant closed 1 month ago

nizwant commented 1 month ago

We decided to split file classes.R, it contained definitions of classes but it became unmanageable. This PR addresses this issue by splitting it into one class per file.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 67.87975% with 203 lines in your changes missing coverage. Please review.

Project coverage is 67.42%. Comparing base (d6d68c3) to head (d9c6c9d).

Files Patch % Lines
R/classes-plate.R 45.76% 173 Missing :warning:
R/classes-sample.R 89.74% 8 Missing :warning:
R/classes-sample_location.R 89.18% 8 Missing :warning:
R/classes-sample_type.R 89.87% 8 Missing :warning:
R/classes-analyte.R 95.58% 3 Missing :warning:
R/helpers.R 78.57% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev #55 +/- ## ========================================== + Coverage 58.63% 67.42% +8.78% ========================================== Files 3 8 +5 Lines 926 1148 +222 ========================================== + Hits 543 774 +231 + Misses 383 374 -9 ```

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