codecov[bot]
commented
4 months ago Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 87.45%. Comparing base (
69458c9) to head (396b2f1).
Additional details and impacted files
```diff @@ Coverage Diff @@ ## main #222 +/- ## ========================================== + Coverage 87.20% 87.45% +0.24% ========================================== Files 21 21 Lines 1767 1770 +3 Branches 321 321 ========================================== + Hits 1541 1548 +7 + Misses 189 186 -3 + Partials 37 36 -1 ```:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
PR Goal?
Merge in some refactoring I was doing while working on something I abandoned since.
Two useful refactoring changes:
web_apy.assemble(): use the input_string directly with a split, instead of converting it to a StringIO just to call readlines() on it.Feedback sought?
regular validation
Priority?
low
Tests added?
Already covered by existing tests.
Confidence?
high
Version change?
no