Skip to content

Make release 1.7.1#390

Merged
apeltzer merged 159 commits intomasterfrom
dev
Apr 17, 2026
Merged

Make release 1.7.1#390
apeltzer merged 159 commits intomasterfrom
dev

Conversation

@atrigila
Copy link
Copy Markdown
Contributor

Closes #382


1.7.1

Added

  • #369 Option to trigger single flowcell without a samplesheet
  • #384 Add nf-tests to functions

Changed

  • #388 Improve functions nf-test.
  • #387 Use nf-schema 2.7.0.
  • #379 Enable workflow outputs.
  • #372 Remove local subworkflows with less than 2 modules.
  • #371 Bump checkqc
  • #354 Bump version and note mgikit addition in 1.7.0 Changelog
  • #355 Template update for nf-core/tools v3.5.1
  • #357 Update MultiQC to 1.33
  • #359 Update FASTP module
  • #363 Use samshee only for validating samplesheet, not for formatting the samplesheet that's used by the demultiplexers.
  • #368 Port nf-tests to UTILS.groovy

Fixed

  • #385 Fix publish statements cannot be mixed with other statements in a dynamic publish path.

Dependencies

Dependency Old version New version
multiqc 1.31 1.33

SPPearce and others added 30 commits October 28, 2025 15:53
Important! Template update for nf-core/tools v3.5.1
Samshee should only validate samplesheet, but we still want BCLconvert to run on the original samplesheet. 

This is for the following reasons: 
 * avoid potential bugs in samshee - if there are any and this leads to a validation error no big harm is done. If it incorrectly rewrites a samplesheet that would be bad
 * easier debugging: we just had such a case where BCLconvert failed on the raw samplesheet but passed on the samshee-formatted one. Took a while to figure out what's going on.
Use unformatted samplesheet for BCLconvert
replace functions tests (`nextflow_workflow`) for `nextflow_function`
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 15, 2026

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

Posted for pipeline commit 257df49

+| ✅ 200 tests passed       |+
#| ❔   1 tests were ignored |#
!| ❗   8 tests had warnings |!
Details

❗ Test warnings:

  • files_exist - File not found: conf/igenomes.config
  • files_exist - File not found: conf/igenomes_ignored.config
  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here
  • pipeline_todos - TODO string in methods_description_template.yml: #Update the HTML below to your preferred methods description, e.g. add publication citation for this pipeline
  • pipeline_todos - TODO string in main.nf: Optionally add in-text citation tools to this list.
  • 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 base.config: Customise requirements for specific processes.

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.2
  • Run at 2026-04-16 15:04:08

@atrigila atrigila requested review from apeltzer and grst April 15, 2026 20:14
@grst grst requested review from julianSchwab and removed request for grst April 16, 2026 06:24
Comment thread CHANGELOG.md
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## v1.7.0
## 1.7.1 - 2026-04-20
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
## 1.7.1 - 2026-04-20
## 1.7.1 - 2026-04-16

Copy link
Copy Markdown
Member

@apeltzer apeltzer left a comment

Choose a reason for hiding this comment

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

LGTM :-) @julianSchwab is running some tests right now

@atrigila atrigila marked this pull request as ready for review April 16, 2026 13:23
@atrigila atrigila requested review from a team, blajoie, nh13 and sam-white04 as code owners April 16, 2026 13:23
@atrigila atrigila requested review from Aratz and removed request for a team April 16, 2026 13:23
@julianSchwab
Copy link
Copy Markdown

julianSchwab commented Apr 17, 2026

Looks good to me, too! Demux runs through fine, MultiQC SAV is there.

Copy link
Copy Markdown

@julianSchwab julianSchwab left a comment

Choose a reason for hiding this comment

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

I checked the changes and compared the results for a flowcell already demuxed with a previous version. Looks good to me as well!

@apeltzer apeltzer merged commit fbec8e4 into master Apr 17, 2026
66 of 68 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.

Make release

10 participants