Closed smoe closed 4 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 80a12ac
+| ✅ 227 tests passed |+
#| ❔ 2 tests were ignored |#
!| ❗ 4 tests had warnings |!
Thanks for contributing @smoe! 😄 Would be nice if you could address my suggestions 🙏
All accepted, very nice, many thanks! Let me quickly rebase this to reduce the number of commits. I presume you are happy with me squashing this all into one.
Hello,
ported this to dev but was running master when I got an error message that was triggered by an empty last line in the sample sheet. Found that only after adding line numbers to the error message and thought that this may be of interest to everyone to have. So, here it comes.
The change are the addition of an iterator and the formatting of the error message's context from "Line" to "Line {}".format(lineNo). For the error message on an invalid number of columns I also added the actual number of columns that were found.
Best, Steffen
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
). Tested the script on a local file.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors). Not required.