Skip to content

Update modules#194

Merged
FernandoDuarteF merged 19 commits intonf-core:devfrom
Eco-Flow:update_modules
Apr 24, 2026
Merged

Update modules#194
FernandoDuarteF merged 19 commits intonf-core:devfrom
Eco-Flow:update_modules

Conversation

@FernandoDuarteF
Copy link
Copy Markdown
Collaborator

@FernandoDuarteF FernandoDuarteF commented Mar 19, 2026

Closes #195.

Changes

  • Updated all modules using nf-core modules update
  • Removed appending versions channel outputs from modules with the new "topics" channel.
  • Fixed linting (although it doesn't seem to work when running the CI checks). Might be worth ignoring these files.

Comments

Some nf-core modules don't have the topics channel to collect the versions. See these bellow. Might be worth migrating these first in the nf-core modules repo before merging this PR.

I will also change the local modules to incorporate the new "topics" channel.

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/genomeqc 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).

@nf-core-bot
Copy link
Copy Markdown
Member

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.5.1.
Please update your pipeline to the latest version.

For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation.

@FernandoDuarteF
Copy link
Copy Markdown
Collaborator Author

FernandoDuarteF commented Mar 19, 2026

Modules in nf-core/modules that have not been updated to topics channel yet:

  • fastavalidator
  • tidk/explore
  • tidk/plot
  • tidk/search
- repeatmasker/repeatmasker
- repeatmasker/builddatabase
- repeatmasker/repeatmodeler

I'm working on fcs/fcsadaptor. See nf-core/modules#10939. The repeatmasker modules are not part of this PR, so we can ignore them for now.

@FernandoDuarteF
Copy link
Copy Markdown
Collaborator Author

I think the EXTRACT_SEQS module can be removed.

@FernandoDuarteF
Copy link
Copy Markdown
Collaborator Author

Same container should be use for the shiny app and the tree plot. I'll make sure this is the case.

@FernandoDuarteF FernandoDuarteF mentioned this pull request Mar 20, 2026
11 tasks
@FernandoDuarteF
Copy link
Copy Markdown
Collaborator Author

The tidk modules and subworkflows have been updated. Removed fastavalidator since tool is deprecated (might be worth looking for another alternative).

@FernandoDuarteF
Copy link
Copy Markdown
Collaborator Author

I think this is ready to merge.

@FernandoDuarteF FernandoDuarteF merged commit 1947a80 into nf-core:dev Apr 24, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update modules

2 participants