Skip to content

Samplesheets settings - #16

Merged
nvnieuwk merged 7 commits into
mainfrom
samplesheets_settings
Sep 24, 2026
Merged

nvnieuwk merged 7 commits into
mainfrom
samplesheets_settings

Conversation

@nvnieuwk

Copy link
Copy Markdown
Member

This pull request introduces a new feature that allows splitting samplesheet outputs by a specified field, enhancing the flexibility of samplesheet creation. It does so by adding a new samplesheets_settings field to the worksheet configuration, supporting a split_by option. The implementation includes updates to the worksheet and samplesheet handling classes, new tests, and documentation updates.

Key changes:

Feature: Configurable Samplesheet Splitting

  • Added a new samplesheets_settings field to worksheet configuration files, supporting a split_by option to control how samplesheets are grouped and written to disk. [1] [2]
  • Implemented the WorksheetSamplesheetsSettings class to validate and store samplesheet settings, including the split_by field.
  • Updated Worksheet and WorksheetSamplesheets classes to accept and use the new settings, splitting samplesheet outputs accordingly when split_by is set. [1] [2] [3]

Testing and Utilities

  • Updated and expanded tests to cover the new samplesheet splitting logic, including changes to test data and snapshot structure. [1] [2] [3] [4] [5] [6] [7]
  • Added a new Utils class with helper methods for test snapshot generation.
  • Adjusted test process logic to handle new output structure. [1] [2]

Documentation

  • Updated CHANGELOG.md and README.md to document the new feature and bump the plugin version to 0.4.0. [1] [2]

@nvnieuwk
nvnieuwk requested a review from matthdsm September 24, 2026 12:57
@nvnieuwk
nvnieuwk merged commit f3b35da into main Sep 24, 2026
8 checks passed
@nvnieuwk
nvnieuwk deleted the samplesheets_settings branch September 24, 2026 15:03
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