Skip to content

Config updates#115

Merged
DLBPointon merged 13 commits intodevfrom
config_updates
Sep 22, 2025
Merged

Config updates#115
DLBPointon merged 13 commits intodevfrom
config_updates

Conversation

@DLBPointon
Copy link
Copy Markdown
Contributor

@DLBPointon DLBPointon commented Sep 16, 2025

Config Updates to match parity with TreeVal

Added an AUTO option for aligner, this way the pipeline will select minimap automatically if genome > 5Gb else bwamem2.

AUTO is now default for ease of use.

Comment out CONDA, pipeline doesn't support it anyway. Turned back on due to merge conflict.

@DLBPointon DLBPointon self-assigned this Sep 16, 2025
@DLBPointon DLBPointon marked this pull request as ready for review September 16, 2025 11:31
@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 16, 2025

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

Posted for pipeline commit b921914

+| ✅ 194 tests passed       |+
#| ❔  32 tests were ignored |#
!| ❗   3 tests had warnings |!
Details

❗ Test warnings:

  • nextflow_config - Config manifest.version should end in dev: 1.5.0
  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here
  • schema_lint - Input mimetype is missing or empty

❔ Tests ignored:

  • files_exist - File is ignored: .github/workflows/awstest.yml
  • files_exist - File is ignored: .github/workflows/awsfulltest.yml
  • files_exist - File is ignored: .github/ISSUE_TEMPLATE/config.yml
  • files_exist - File is ignored: assets/multiqc_config.yml
  • files_exist - File is ignored: assets/nf-core-curationpretext_logo_light.png
  • files_exist - File is ignored: docs/images/nf-core-curationpretext_logo_light.png
  • files_exist - File is ignored: docs/images/nf-core-curationpretext_logo_dark.png
  • files_exist - File is ignored: conf/igenomes.config
  • files_exist - File is ignored: conf/igenomes_ignored.config
  • files_exist - File is ignored: CODE_OF_CONDUCT.md
  • nextflow_config - Config variable ignored: manifest.name
  • nextflow_config - Config variable ignored: manifest.homePage
  • files_unchanged - File does not exist: CODE_OF_CONDUCT.md
  • files_unchanged - File ignored due to lint config: LICENSE or LICENSE.md or LICENCE or LICENCE.md
  • files_unchanged - File ignored due to lint config: .github/CONTRIBUTING.md
  • files_unchanged - File ignored due to lint config: .github/ISSUE_TEMPLATE/bug_report.yml
  • files_unchanged - File ignored due to lint config: .github/ISSUE_TEMPLATE/config.yml
  • files_unchanged - File ignored due to lint config: .github/ISSUE_TEMPLATE/feature_request.yml
  • files_unchanged - File ignored due to lint config: .github/PULL_REQUEST_TEMPLATE.md
  • files_unchanged - File ignored due to lint config: .github/workflows/branch.yml
  • files_unchanged - File ignored due to lint config: .github/workflows/linting_comment.yml
  • files_unchanged - File ignored due to lint config: .github/workflows/linting.yml
  • files_unchanged - File ignored due to lint config: assets/email_template.html
  • files_unchanged - File ignored due to lint config: assets/email_template.txt
  • files_unchanged - File ignored due to lint config: assets/sendmail_template.txt
  • files_unchanged - File does not exist: assets/nf-core-curationpretext_logo_light.png
  • files_unchanged - File does not exist: docs/images/nf-core-curationpretext_logo_light.png
  • files_unchanged - File does not exist: docs/images/nf-core-curationpretext_logo_dark.png
  • files_unchanged - File ignored due to lint config: docs/README.md
  • files_unchanged - File ignored due to lint config: .gitignore or .prettierignore
  • actions_awstest - 'awstest.yml' workflow not found: /home/runner/work/curationpretext/curationpretext/.github/workflows/awstest.yml
  • multiqc_config - multiqc_config

✅ Tests passed:

Run details

  • nf-core/tools version 3.3.2
  • Run at 2025-09-19 12:52:02

@yumisims yumisims self-requested a review September 16, 2025 14:56
@yumisims
Copy link
Copy Markdown
Contributor

should we discuss in the meeting whether should use auto? I think we should just default to minimap2, keep the option open for curators if they want to use bwamem2?

@DLBPointon
Copy link
Copy Markdown
Contributor Author

@yumisims
I've confirmed with Jo that AUTO is fine to be left as the default. Some more in depth testing can be done later.

I'm just updating a couple of docs related things then this will be good to review.

Copy link
Copy Markdown
Contributor

@prototaxites prototaxites left a comment

Choose a reason for hiding this comment

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

LGTM!

@DLBPointon DLBPointon merged commit 2f32cb3 into dev Sep 22, 2025
7 checks passed
@DLBPointon DLBPointon deleted the config_updates branch October 3, 2025 11: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