Skip to content

Add exit status for RCLONE modules in multiqc report - #79

Merged
delfiterradas merged 8 commits into
nf-core:devfrom
delfiterradas:dev
Aug 7, 2026
Merged

Add exit status for RCLONE modules in multiqc report#79
delfiterradas merged 8 commits into
nf-core:devfrom
delfiterradas:dev

Conversation

@delfiterradas

@delfiterradas delfiterradas commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Closes #77

PR checklist

  • 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 you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/datasync branch on the nf-core/test-datasets repository.
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@delfiterradas delfiterradas self-assigned this Aug 7, 2026
@github-actions

github-actions Bot commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 7d1f26d

+| ✅ 208 tests passed       |+
#| ❔   4 tests were ignored |#
#| ❔   1 tests had warnings |#
!| ❗   2 tests had warnings |!
Details

❗ Test warnings:

  • readme - README contains the placeholder zenodo.XXXXXXX. This should be replaced with the zenodo doi (after the first release).
  • pipeline_todos - TODO string in README.md: Add citation for pipeline after first release. Uncomment lines below and update Zenodo doi and badge at the top of this file.

❔ Tests ignored:

❔ Tests fixed:

✅ Tests passed:

Run details

  • nf-core/tools version 4.1.0
  • Run at 2026-08-07 19:41:03

@github-actions

github-actions Bot commented Aug 7, 2026 •
edited
Loading

Copy link
Copy Markdown

❌ nf-test failed with latest Nextflow version

Note

Tests with Nextflow's latest version failed but it will not cause a CI workflow failure.
Please check if the failure is expected with newer (edge-)releases of Nextflow or if it needs fixing.

  • ❌ docker | latest-everything | Shard 2/5

See the full run for details.

@delfiterradas

delfiterradas commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

This is what the multiqc report looks like: multiqc_report.html

@delfiterradas
delfiterradas marked this pull request as ready for review August 7, 2026 17:18
@delfiterradas
delfiterradas requested a review from atrigila August 7, 2026 17:30
atrigila
atrigila approved these changes Aug 7, 2026
View reviewed changes

@atrigila atrigila left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me! Can you add a small section in output.md documenting this new section's info as well?

@delfiterradas
delfiterradas requested a review from atrigila August 7, 2026 20:25
@delfiterradas
delfiterradas merged commit 5ce0a10 into nf-core:dev Aug 7, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@atrigila atrigila Awaiting requested review from atrigila

Assignees

@delfiterradas delfiterradas

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add exit status for RCLONE modules in multiqc report

2 participants

@delfiterradas @atrigila