Fix error in output for contents #73
Conversation
This is a better solution for openpathsampling#71 (should replace openpathsampling#72). We really don't want all that junk in the `contents` output anyway. The purpose of that line to give the filename as part of the output, so let's just give the filename.
Codecov ReportBase: 100.00% // Head: 99.96% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #73 +/- ##
===========================================
- Coverage 100.00% 99.96% -0.04%
===========================================
Files 110 110
Lines 5415 5418 +3
===========================================
+ Hits 5415 5416 +1
- Misses 0 2 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
|
This looks good to me. Please either add a test for the added error, or add a |
sroet
left a comment
sroet
left a comment
There was a problem hiding this comment.
LGTM! codeclimate issue is not relevant for this temporary fix in my opinion. Feel free to merge
2 participants
This is a better solution for #71 (should replace #72).
We really don't want all that junk in the
contentsoutput anyway. The purpose of that line to give the filename as part of the output, so let's just give the filename.