Closed eumiro closed 3 years ago
Thanks for your contribution!
We had a short discussion about the usage of f-strings in https://github.com/koodaamo/tnefparse/pull/79#discussion_r528839782 and as @petri merged the PR (with added f-strings), I think there is a consensus now to go with f-strings :-)
Quite some lines you changed are currently without coverage.
It would be awesome to add the missing coverage with this PR.
And also, as @petri suggested in another issue, please add a change log entry.
Codecov Report
80.98% <0.00%> (ø)
92.83% <33.33%> (ø)
80.95% <50.00%> (ø)
96.25% <100.00%> (ø)
Continue to review full report at Codecov.