Skip to content

Changing test_full.config to use the larger test dataset from viralrecon#54

Merged
maxulysse merged 3 commits intodevfrom
test_full_update
Mar 11, 2026
Merged

Changing test_full.config to use the larger test dataset from viralrecon#54
maxulysse merged 3 commits intodevfrom
test_full_update

Conversation

@jen-reeve
Copy link
Copy Markdown

@jen-reeve jen-reeve commented Mar 10, 2026

This PR changes the dataset provided to test_full.config so it is no longer the same as the dataset provided in the test.config. To keep things simple I have used the full test dataset from viralrecon as the previous dataset was the smaller test dataset from viralrecon.

In addition to confirming that I could run the test profile (with Singularity rather than Docker in the Codespaces instance), I also confirmed that the test_full profile ran in standard and debug modes.

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • If necessary, also make a PR on the nf-core/demo branch on the nf-core/test-datasets repository.
  • 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>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 10, 2026

Warning

Newer version of the nf-core template is available.

Your pipeline is using an old version of the nf-core template: 3.5.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.

@jen-reeve jen-reeve moved this to In progress in Hackathon March 2026 Mar 10, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 10, 2026

nf-core pipelines lint overall result: Passed ✅

Posted for pipeline commit 8b48bb6

+| ✅ 202 tests passed       |+
Details

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.1
  • Run at 2026-03-11 07:46:30

@jen-reeve jen-reeve linked an issue Mar 10, 2026 that may be closed by this pull request
@jen-reeve jen-reeve marked this pull request as ready for review March 10, 2026 22:12
@jen-reeve jen-reeve moved this from In progress to Ready for review in Hackathon March 2026 Mar 10, 2026
@jen-reeve jen-reeve added the enhancement New feature or request label Mar 10, 2026
@maxulysse maxulysse changed the base branch from master to dev March 11, 2026 07:44
@maxulysse maxulysse merged commit ed969ce into dev Mar 11, 2026
7 checks passed
@maxulysse maxulysse deleted the test_full_update branch March 11, 2026 07:46
@jen-reeve jen-reeve moved this from Ready for review to Done in Hackathon March 2026 Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Provide actual full sized test to test_full

2 participants