Skip to content

Conversation

@Aratz
Copy link
Contributor

@Aratz Aratz commented Nov 10, 2025

This PR copies the samplesheet to the output directory at the beginning of the run.

Leaving this as draft for now as it needs to be merged after #163

PR checklist

  • This comment contains a description of changes (with reason).
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.

@Aratz Aratz self-assigned this Nov 10, 2025
@Aratz Aratz force-pushed the feature/pna-1351_publish_samplesheet branch from 2508c6e to de7baea Compare November 10, 2025 12:07
@Aratz Aratz force-pushed the feature/pna-1351_publish_samplesheet branch from de7baea to 5843b9f Compare November 21, 2025 09:08
@nf-core-bot
Copy link
Member

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.4.1.
Please update your pipeline to the latest version.

For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation.

@Aratz Aratz marked this pull request as ready for review November 21, 2025 09:26
@Aratz Aratz requested a review from a team as a code owner November 21, 2025 09:26
@Aratz Aratz requested a review from Copilot November 21, 2025 09:27
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds functionality to copy the input samplesheet to the output directory's pipeline_info folder at the start of the pipeline run, improving reproducibility by preserving the exact input configuration used.

Key Changes:

  • Copies the samplesheet file to the output directory at pipeline initialization
  • Updates test snapshots to include the samplesheet in expected outputs
  • Documents the new output in the pipeline documentation

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
workflows/pixelator.nf Adds file copy operation to publish samplesheet to pipeline_info directory
tests/pna.nf.test.snap Updates test snapshots to include samplesheet file in expected outputs
tests/mpx.nf.test.snap Updates test snapshots to include samplesheet file in expected outputs
docs/output.md Documents the samplesheet as a new output artifact
CHANGELOG.md Records the feature addition in the changelog

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Aratz Aratz merged commit c83d71a into nf-core:dev Nov 21, 2025
16 checks passed
@Aratz Aratz mentioned this pull request Dec 11, 2025
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