+
Skip to content

Release 2.0.0 #184

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 193 commits into from
Apr 10, 2025
Merged

Release 2.0.0 #184

merged 193 commits into from
Apr 10, 2025

Conversation

scwatts
Copy link
Collaborator

@scwatts scwatts commented Apr 10, 2025

No description provided.

scwatts and others added 30 commits February 1, 2024 15:15
* add source selection for RNA BAM (samplesheet, alignment subworkflow)
* use appropriate function call to determine present of RNA
* relocate isofox_read_length assignment (wgts and targeted workflows)
- allow custom panels with config 'force_panel'
- UMI duplex UMI delimiter with config 'umi_duplex_delim'
- UMI extraction from fastq with config 'umi_length'
Copy link

github-actions bot commented Apr 10, 2025

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

Posted for pipeline commit 9f93f51

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

❗ Test warnings:

  • files_exist - File not found: assets/multiqc_config.yml
  • readme - README contains the placeholder zenodo.XXXXXXX. This should be replaced with the zenodo doi (after the first release).
  • 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!
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • 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.0.2
  • Run at 2025-04-10 04:47:50

@scwatts scwatts merged commit d1218d2 into master Apr 10, 2025
20 checks passed
@jfy133
Copy link
Member

jfy133 commented Apr 10, 2025

@scwatts quick question - is this PR going for an official release?

If so, while it's great you got two reviews - we strongly encourage that at least one of the reviewers is 'external' from the project/same employer. This is to ensure that an 'isolated bubble' forms of styles/patterns starts forming. This has happened in the past and the results were not pretty (pipelines very diverged from template/guidelines etc, which caused knock-on effects downstream in future template updates/reviews etc).

If you've not done the actual GitHub release yet, it would be great if you could ask on #release-review-trading to trade with someone else to help keep things in check (And use that in the future)

@ohofmann
Copy link

Following up on @jfy133 's comment so it doesn't get lost -- we missed the requirement for an external reviewer in this PR review process. Absolutely makes sense and we'll work with the nf-core community to have a reviewer or two on standby (and contribute to other reviews in the meantime, about to join the review-trading channel). Apologies again!

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.

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载