Skip to content

Add ampliseq pipeline support - #383

Merged
maxulysse merged 7 commits into
devfrom
feature/add-ampliseq-pipeline
Aug 20, 2026
Merged

Add ampliseq pipeline support#383
maxulysse merged 7 commits into
devfrom
feature/add-ampliseq-pipeline

Conversation

@maxulysse

@maxulysse maxulysse commented Aug 13, 2026

Copy link
Copy Markdown
Member

Adds ampliseq pipeline support to fetchngs.

Replace and close:

Changes

  • Add ampliseq to nextflow_schema.json enum
  • Add ampliseq to pipeline_extras map with run column
  • Add test for ampliseq pipeline

PR checklist

  • This comment contains a description of changes (with reason)
  • CHANGELOG.md has been updated

Generated by opencode
Verified by @maxulysse

Generated by opencode
Generated by opencode
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown

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

Posted for pipeline commit 2abb910

+| ✅ 181 tests passed       |+
#| ❔  13 tests were ignored |#
!| ❗   2 tests had warnings |!
Details

❗ Test warnings:

❔ Tests ignored:

  • files_exist - File is ignored: .github/workflows/awsfulltest.yml
  • files_exist - File is ignored: .github/workflows/awstest.yml
  • files_exist - File is ignored: assets/multiqc_config.yml
  • files_exist - File is ignored: conf/igenomes.config
  • files_exist - File is ignored: conf/igenomes_ignored.config
  • files_exist - File is ignored: conf/modules.config
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: assets/sendmail_template.txt
  • files_unchanged - File ignored due to lint config: assets/nf-core-fetchngs_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-fetchngs_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-fetchngs_logo_dark.png
  • multiqc_config - multiqc_config
  • modules_config - modules_config

✅ Tests passed:

Run details

  • nf-core/tools version 4.0.3
  • Run at 2026-08-17 07:17:11

@d4straub d4straub left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@maxulysse
maxulysse merged commit a1238b3 into dev Aug 20, 2026
22 checks passed
@maxulysse
maxulysse deleted the feature/add-ampliseq-pipeline branch August 20, 2026 09:30
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.

2 participants