sourmash compute nucleotide was getting skipped but not peptide - fix the logic for it

pranathivemuri commented 4 years ago

Many thanks to contributing to nf-core/kmermaid!

Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).

PR checklist

[ ] PR is to dev rather than master
[ ] This comment contains a description of changes (with reason)
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If necessary, also make a PR on the nf-core/kmermaid branch on the nf-core/test-datasets repo
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[ ] Make sure your code lints (nf-core lint .).
[ ] Documentation in docs is updated
[ ] CHANGELOG.md is updated
[ ] README.md is updated

pranathivemuri commented 4 years ago

@olgabot could you please review this PR?

pranathivemuri commented 4 years ago

@olgabot could you please review this PR?

olgabot commented 4 years ago

Thanks for the ping!

pranathivemuri commented 4 years ago

@olgabot since the markdown bits don't completely pass, I can't merge. if you have the rights to merge, could you please?