Closed nschcolnicov closed 1 month ago
nf-core pipelines lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit debb5a7
+| ✅ 242 tests passed |+
#| ❔ 1 tests were ignored |#
!| ❗ 3 tests had warnings |!
Besides pulling from the template, I also added a new config profile for setting the resources in CI tests, since I had to remove the max resources params and the checkmax function with the new template I also had to update the .stat file snaps because after having to update the resource definitions for the pipeline, it affects the alignment results.
nf-core linting seems to be failing due to an issue with the new template that is not picking up the igenomes_base value from the ignore params array. I have already reported this issue, I'll update the PR once this is fixed
Hm, not sure I find the issue on tools @nschcolnicov which ticket did you create?
Hi @apeltzer I didn't, I reached out on slack https://nfcore.slack.com/archives/CE6PELWR4/p1728478362119159
The issue has already been fixed in dev, and will be released soon: https://nfcore.slack.com/archives/CE6PELWR4/p1728478362119159 On the meantime @atrigila fixed this issue temporarily by adding the ignore params to the nextflow_schema in case we want to merge this ASAP
@atrigila We will be merging this, while we wait for nf-core to merge the fix for the ignore params issue, for now we added the igenomes_base to the schema instead of ignoring it, we can simply remove it once its fixed
Sorry for the direct commit; I thought that it would materialize as a simple suggestion. I will check the GitHub docs in more details…
PR checklist
nf-core lint
).nf-test test main.nf.test -profile test,docker
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).