-
Notifications
You must be signed in to change notification settings - Fork 190
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
@ewels implementation of nf-core/tools#936 #943
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #943 +/- ##
==========================================
+ Coverage 69.56% 69.61% +0.05%
==========================================
Files 34 34
Lines 4173 4180 +7
==========================================
+ Hits 2903 2910 +7
Misses 1270 1270
Continue to review full report at Codecov.
|
add sphinx to requirements file
Rebased to hide my lack of local testing. Hopefully Kevin doesn't read his GitHub email notifications.. 👀
8cfcb81
to
396ebc2
Compare
What the hell have I done here 🤦🏻 |
Ok I need to go sleep because I am doing super stupid stuff. Pushed as a commit to #926 instead |
😆 |
My version of #936 - have a look @KevinMenden and see what you think.
Main differences:
UnicodeDecodeError
and copies file instead of exiting with an errorPR checklist
CHANGELOG.md
is updateddocs
is updated