WIP: Import taxpasta #574
WIP: Import taxpasta #574
Conversation
Midnighter
force-pushed
the
import-taxpasta
branch
from
3053df2
to
a0c1a5c
Compare
|
Thank you for the detailed review @TuomasBorman! I have some open questions from your comments and will appreciate your assistance with the documentation. It's not clear to me why to break up the docstring like that for a single function and how to direct |
|
Hi @TuomasBorman, Starting next week, I will have more time again and could try to complete this PR. I would need some advice and decisions from your side, though, please. If you can take another look, I will be grateful. |
Alright, sounds good. I have also been busy, but I will check during this week. |
|
@TuomasBorman, I believe, I have addressed all of your comments. Please have another look and let me know if there's something else to be addressed. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## devel #574 +/- ##
========================================
Coverage ? 68.04%
========================================
Files ? 43
Lines ? 5194
Branches ? 0
========================================
Hits ? 3534
Misses ? 1660
Partials ? 0 ☔ View full report in Codecov by Sentry. |
To do:
biomformat:::. I see no way around this at the moment.