Closed DLBPointon closed 8 months ago
Currently relies on PR #34 to be merged first
black
) is failingTo keep the code consistent with lots of contributors, we run automated code consistency checks. To fix this CI test, please run:
black
: pip install black
black .
Once you push these changes the test should pass, and you can hide this comment :+1:
We highly recommend setting up Black in your code editor so that this formatting is done automatically on save. Ask about it on Slack for help!
Thanks again for your contribution!
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 1f3d386
+| ✅ 125 tests passed |+
#| ❔ 21 tests were ignored |#
!| ❗ 21 tests had warnings |!
Local testing has revealed an error, with a slightly larger data set causing an NCBI C++ error.
could Eerik to have a check on this workflow?
could Eerik to have a check on this workflow?
That's fine, I just wanted to chase down this error with BLASTN first. I'll send the error on Slack, hopefully someone understands it a bit more.
I have PR to fix the blastn nf core module, could use it here instead of creating a new module, currently the PR is waiting for nf-core to fix conda problem then merge. https://github.com/nf-core/modules/pull/4245#issuecomment-1782393384
I have PR to fix the blastn nf core module, could use it here instead of creating a new module, currently the PR is waiting for nf-core to fix conda problem then merge. nf-core/modules#4245 (comment)
PR waiting on above fix by nf-core + nf-test
Adds blast for organelles