Skip to content

Release 2.1.0#215

Merged
scwatts merged 98 commits into
masterfrom
dev
Jun 30, 2025
Merged

Release 2.1.0#215
scwatts merged 98 commits into
masterfrom
dev

Conversation

@scwatts
Copy link
Copy Markdown
Member

@scwatts scwatts commented Jun 11, 2025

No description provided.

scwatts and others added 30 commits November 6, 2024 16:08
Additionally, switch to Bioconda package and BioContainers images
- the fastp split_by_lines argument value is calculated from max_fastq_records
- max_fastq_records values greater or equal to 536,870,912 (2 ^ 32 / 2 / 4) cause an integer overflow
- this is fixed by casting max_fastq_records to Long type before calculating the value for split_by_lines
Fix integer overflow of fastp --split_by_lines value
Fix REDUX TSV discovery for non-local files
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 11, 2025

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

Posted for pipeline commit 1282615

+| ✅ 212 tests passed       |+
#| ❔  10 tests were ignored |#
!| ❗  16 tests had warnings |!
Details

❗ Test warnings:

  • files_exist - File not found: assets/multiqc_config.yml
  • files_unchanged - LICENSE does not match the template
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • pipeline_todos - TODO string in main.nf: Optionally add bibliographic entries to this list.
  • pipeline_todos - TODO string in main.nf: Only uncomment below if logic in toolCitationText/toolBibliographyText has been filled!
  • schema_params - Schema param panel not found from nextflow config
  • schema_params - Schema param fastp_umi_location not found from nextflow config
  • schema_params - Schema param fastp_umi_length not found from nextflow config
  • schema_params - Schema param fastp_umi_skip not found from nextflow config
  • schema_params - Schema param redux_umi_duplex_delim not found from nextflow config
  • schema_params - Schema param genome_version not found from nextflow config
  • schema_params - Schema param genome_type not found from nextflow config
  • schema_params - Schema param ref_data_hmf_data_path not found from nextflow config
  • schema_params - Schema param ref_data_panel_data_path not found from nextflow config
  • schema_params - Schema param ref_data_genome_gtf not found from nextflow config
  • schema_params - Schema param ref_data_hla_slice_bed not found from nextflow config

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 3.3.1
  • Run at 2025-06-30 02:39:40

Comment thread nextflow.config
Comment thread nextflow.config
Comment thread docs/usage.md Outdated
Copy link
Copy Markdown
Member

@maxulysse maxulysse left a comment

Choose a reason for hiding this comment

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

minor comments

Comment thread docs/images/oncoanalyser_pipeline.png
Comment thread nextflow.config
Comment thread nf-test.config
Copy link
Copy Markdown
Member

@maxulysse maxulysse left a comment

Choose a reason for hiding this comment

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

Looking good.

Still some deviations from the nf-core standards, but nothing blocking a release from me.

Comment thread CHANGELOG.md Outdated
@scwatts scwatts requested a review from charlesshale June 30, 2025 02:37
@scwatts scwatts merged commit 0d0dc25 into master Jun 30, 2025
17 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.

5 participants