Skip to content

Migrate to channel topics#133

Merged
fasterius merged 22 commits intodevfrom
topics
Feb 12, 2026
Merged

Migrate to channel topics#133
fasterius merged 22 commits intodevfrom
topics

Conversation

@fasterius
Copy link
Collaborator

@fasterius fasterius commented Jan 27, 2026

This PR adds usage of channel topics for the pipeline. Currently waiting for nf-core/modules#9787 (SPACERANGER_COUNT) and nf-core/modules#9749 (UNTAR) module PRs to be merged before this PR can be finished.

PR checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • CHANGELOG.md is updated.

@fasterius fasterius changed the title Topics Migrate to channel topics Jan 27, 2026
@github-actions
Copy link

github-actions bot commented Jan 27, 2026

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

Posted for pipeline commit 4c09a31

+| ✅ 214 tests passed       |+
#| ❔   8 tests were ignored |#
!| ❗   6 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 nextflow.config: Optionally, you can add a pipeline-specific nf-core config at https://github.com/nf-core/configs
  • 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.
  • pipeline_todos - TODO string in README.md: Add bibliography of tools and data used in your pipeline
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here

❔ Tests ignored:

  • files_exist - File is ignored: conf/igenomes.config
  • files_exist - File is ignored: conf/igenomes_ignored.config
  • files_unchanged - File ignored due to lint config: .gitattributes
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: assets/nf-core-spatialvi_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-spatialvi_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-spatialvi_logo_dark.png
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.1
  • Run at 2026-02-12 09:02:15

@fasterius
Copy link
Collaborator Author

UNTAR is included in another PR: nf-core/modules#9796.

@nf-core-bot
Copy link
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.

@fasterius
Copy link
Collaborator Author

fasterius commented Feb 6, 2026

@cavenel Did something change in the Visium HD tests? It's running out of time, and the latest commits I see for the test_spaceranger_hd_ffpe were 5 months ago.

@cavenel
Copy link
Collaborator

cavenel commented Feb 11, 2026

Except from the version of SpaceRanger I don't think anything changed... Maybe the running time is around 1h and sometimes take longer? I guess we could just increase to 2h for this test.

3 weeks ago the whole test 3/4 took 38 minutes, so quite far from 1h. I tried to run it again, we will see.

@fasterius fasterius marked this pull request as ready for review February 11, 2026 14:05
@fasterius fasterius merged commit a4bd0e6 into dev Feb 12, 2026
15 checks passed
@fasterius fasterius deleted the topics branch February 12, 2026 13:06
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.

3 participants