Skip to content
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

Bulk update meta yml modules #6015

Draft
wants to merge 120 commits into
base: batch_update_staging
Choose a base branch
from

Conversation

mirpedrol
Copy link
Member

Close #4983
Close #5830

Updates the meta.yml of modules to the correct structure by using nf-core/tools after implementing nf-core/tools#3028 and nf-core/tools#3032
The changes were generated by running nf-core modules lint --all --fix

Needs updating the JSON schema which validates meta.yml files: #5837

Same changes should be implemented for subworkflows.

Note

Open as a draft PR to avoid requresting review to code owners.

jfy133 and others added 30 commits June 19, 2024 19:37
…l and add stub (nf-core#5849)

* Add missing optional input channel to abricate

* Update abricate module to include missing optional input path

* Update modules/nf-core/abricate/run/tests/main.nf.test

Co-authored-by: Carson J Miller <[email protected]>

* Fix linting

* Update modules/nf-core/abricate/run/main.nf

* Get downstream module test

---------

Co-authored-by: Carson J Miller <[email protected]>
* Update environment.yml

* Update main.nf
* update to samtools 1.20

* keep samtools 1.19.2 for now

* update snapshot

* [automated] Fix linting with Prettier

* keep 1.20 afterall

* update snapshot

---------

Co-authored-by: nf-core-bot <[email protected]>
* update recipe

* update rthreads

* update tests

* Update meta.yml
….10 (nf-core#5857)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* add unoutput option

* trying to fix emit

* CI fixed, tests still incorrect

* Fixed the checksums

* Fixed the syntax

* Call this "unselected" rather than "unoutput"

* Alignment rocks

* bugfix: the variable needs to be visible by the output declaration

* bugfix: this was not the regular file name

* Updated the test case

* fixup! Alignment rocks

* Updated the meta.yml

---------

Co-authored-by: Matthieu Muffato <[email protected]>
* stubs and test for bbmap_bbsplit

* add test for index

* add stubs for custom/catadditionalfasta

* add stubs for fq/subsample

* add stubs for hisat2/extractsplicesites

* update fastq_align_hisat2 snapshot

* add stubs for hisat2/build

* add stubs for hisat2/align

* more stubs bbmap/bbsplit

* add stubs for preseq/lcextrap

* more stubs for fq/subsample

* proper stubs for hisat2/align

* remove old snaps
* Bump concoct/cutupfasta version, add stub, add nf-test

* Fix linting
* Adding meta to multivcfanalyzer, updating test to nf-test

* Apply suggestions from code review

Co-authored-by: James A. Fellows Yates <[email protected]>

* fix meta.yml

* Add sort to vcfs

* Add tab to tags

* add checks for files due to failing md5 sums across docker,conda,singularity

---------

Co-authored-by: James A. Fellows Yates <[email protected]>
Co-authored-by: Simon Pearce <[email protected]>
* Bump concoct/cutupfasta version, add stub, add nf-test

* Bump conda build, add stub and nf-test for CONCOT/concoctcoveragetable

* Remove pytest entry

---------

Co-authored-by: Aida Andrades Valtueña <[email protected]>
* Added stub logic and test to fastq_fastqc_umitools_fastp

* Now emitting trimmed reads
…output (nf-core#5850)

* Enable single fastq.gz-output from spring/decompress by setting meta.single_end or meta.one_strand

* Controlling output with value-channel write_one_fastq_gz

* Update modules/nf-core/spring/decompress/meta.yml

Co-authored-by: Maxime U Garcia <[email protected]>

* nf-test for spring/decompress

* Trying to please linter

* Trying to please linter

---------

Co-authored-by: Simon Pearce <[email protected]>
Co-authored-by: Maxime U Garcia <[email protected]>
* everything but snapshot

* fix gz snap

* update testdata paths

* Update main.nf.test

* update test

---------

Co-authored-by: Simon Pearce <[email protected]>
mapAD: fix `version.yml` emission
* add stub and stub test to chromap, change test names

* add stub and stub test to chromap, change test names

* change setup

* move setup out of test

* move
…a to v0.28.6 (nf-core#5878)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update main.nf 

VEP html report is missing when publishing results because by default vep html file is `_summary.html` instead of `.summary.html`. This changes just adjust the module to the vep defaults to avoid the issue.

* Updated html pattern in VEP to be more generic
* Bump shinyngs modules

* Bump versions for snapshot
…e#5869)

* Version update, stub, nf-test for concoct/concoct

* Bump version, add nf-test, stub to concoct/mergecutupclustering

* nf-test, stub, and version conda build version bump for concoct/extrafastabins

* Remove a superfluous config and specify a used one

* Update modules/nf-core/concoct/extractfastabins/main.nf

Co-authored-by: Simon Pearce <[email protected]>

* Remove superfluous config file

* Clean up code

* Repair all stub tests that were empty

---------

Co-authored-by: Simon Pearce <[email protected]>
* fix: do not fail on mismatch for samples not binned with concoct

* Migrate to pytest

* Improved Harshil alignment

---------

Co-authored-by: James A. Fellows Yates <[email protected]>
* Generated structure for new module

* Implemented module

* Added semi-functional tests

* Cleaned tests - test don't work yet

* fix main, start nf-test

* add new test data

* fix linting

* try to assert folders

* fix tests

* change output to prefix

* change to prefix

* update to prefix

---------

Co-authored-by: vickylaram <[email protected]>
…uated (nf-core#5887)

Fix bug causing output name to be incorrectly evaluated
* nf-test for snpsites

* remove old pytest

* Delete modules/nf-core/snpsites/tests/nextflow.config

* Update modules/nf-core/snpsites/tests/main.nf.test

Co-authored-by: Joon Klaps <[email protected]>

* Update assert

Co-authored-by: Joon Klaps <[email protected]>

* renamed test

Co-authored-by: Joon Klaps <[email protected]>

* reverted the input

* updated input...again

Co-authored-by: Joon Klaps <[email protected]>

* fixed typo

Co-authored-by: Joon Klaps <[email protected]>

* Regenerate snapshot and fix linting

---------

Co-authored-by: Joon Klaps <[email protected]>
Co-authored-by: Simon Pearce <[email protected]>
….toString() - '.gz'` (nf-core#5879)

* Added prefix to gunzip

* Fixed linting issues in fargene and leviosam2/lift

* Removed unstable liftoff/conda
…ultiqc) (nf-core#5501)

* created new output channel capturing stderr of trimmomatic

* updated output channel for trimmomatic

* Add stub test and update snapshots

* Add stub and update tests

* Fix linting and add tee

---------

Co-authored-by: John Palmer <[email protected]>
Co-authored-by: Simon Pearce <[email protected]>
Co-authored-by: James A. Fellows Yates <[email protected]>
GallVp and others added 25 commits July 15, 2024 22:06
* Added agat/spmergeannotations

* Added tests with config
* Edit content with subworkflow name

* Rename subworkflow folder

* Readd subworkflow with old name, with deprecation

* remove rogue word

* Appease eclint

* Remove test suite for deprecated subworkflow
…ore#5823)

* Using seqera containers for sentieon

* update containers for sentieon modules with quay.io registry

---------

Co-authored-by: Simon Pearce <[email protected]>
Co-authored-by: Adam Talbot <[email protected]>
Co-authored-by: maxulysse <[email protected]>
* unzip stub, nf-test, snap

* Revert "unzip stub, nf-test, snap"

This reverts commit dec35aa.

* Reapply "unzip stub, nf-test, snap"

This reverts commit e0d708e.

* added minimal required tests for process and stub

* added snapshot

* simplify snapshot input in stub test

Co-authored-by: Matthias Hörtenhuber <[email protected]>

* remove unneeded stub files

* migrate pytest for unzipfiles

* add snapshot file for script block

* added stub

* added stub and stub test

* updated stub snapshot for unzip

* remove unneeded test config

* Update snaps and malt

* Update malt for linting

* Fix linting for maltextract

* Swap test-data paths for malt modules

* Update modules/nf-core/maltextract/tests/main.nf.test

---------

Co-authored-by: Sateesh <[email protected]>
Co-authored-by: Jenny Leopoldina Smith <[email protected]>
Co-authored-by: Matthias Hörtenhuber <[email protected]>
Co-authored-by: Maxime U Garcia <[email protected]>
…5982)

* Improve strandedness devivation

* Add missing params to nf-test
…FASTAs, port tests (nf-core#5988)

* rrna sequences make more sense as a channel of FASTAs

* Also update tests

* Add function test to swf

* Fix up subworkflow function testing
…ules (nf-core#5989)

* add meta2 to input fasta channel

* update snapshot; nothing new but timestamp

* Update modules/nf-core/lofreq/viterbi/tests/main.nf.test

Co-authored-by: Felix Lenner <[email protected]>

* Update modules/nf-core/lofreq/viterbi/tests/main.nf.test

Co-authored-by: Felix Lenner <[email protected]>

* Update modules/nf-core/lofreq/viterbi/tests/main.nf.test

Co-authored-by: Felix Lenner <[email protected]>

---------

Co-authored-by: Felix Lenner <[email protected]>
* Initial commit

* Rerun nf-test

* Change assertion for duplex_qc output

* Add ggplot2 to version.yml

* Update snapshot

* Correct conda packages version
* add-usearch-unoise3

* add seqfu/derep

* Delete modules/nf-core/usearch/unoise3/environment.yml

* Delete modules/nf-core/usearch/unoise3/main.nf

* remove files

* update metadata

* Update modules/nf-core/seqfu/derep/main.nf

Co-authored-by: Simon Pearce <[email protected]>

* Update modules/nf-core/seqfu/derep/main.nf

Co-authored-by: Simon Pearce <[email protected]>

* Update modules/nf-core/seqfu/derep/main.nf

Co-authored-by: Simon Pearce <[email protected]>

* Update modules/nf-core/seqfu/derep/main.nf

Co-authored-by: Simon Pearce <[email protected]>

* Update modules/nf-core/seqfu/derep/main.nf

Co-authored-by: Simon Pearce <[email protected]>

* Update modules/nf-core/seqfu/derep/tests/main.nf.test

Co-authored-by: Simon Pearce <[email protected]>

* Fix dataset in -stub

* Clarify description

* Update modules/nf-core/seqfu/derep/meta.yml

Co-authored-by: Simon Pearce <[email protected]>

* check for pre-existing output file && update test

* Update main.nf

---------

Co-authored-by: Simon Pearce <[email protected]>
* liftoff: Readded conda environment

* Added liftoff to env file

* Cleaned up env file
….3 (nf-core#5996)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Fix output naming, add req argument to input

* add new input to nf-test

* update nf-test due to changes in upstream module inputs

* update snapshots

* add metas to other input channels

* update tests according to changes in module inputs.

* Apply suggestions from code review

Co-authored-by: James A. Fellows Yates <[email protected]>

* rename elongation factor for consistency with other CM module

---------

Co-authored-by: James A. Fellows Yates <[email protected]>
* Fix output naming, add req argument to input

* add new input to nf-test

* update nf-test due to changes in upstream module inputs

* update snapshots

* add metas to other input channels

* update tests according to changes in module inputs.

* add meta to elongation factor channel for consistency with other CM module

* update nf-test to include new meta

* rename metas
* Add stub and test to lofreq/indelqual

* Add author

* new test setup using --migrate-pytest and PR feedback

* Update main.nf.test

* Update meta.yml

* Update main.nf

* add snap; del pytest

* Update meta.yml

* Update main.nf.test

* Update meta.yml

* update test

* minor fix in stub test
* Update tabix and add threads

* fix linting
* refactor: Clean up the logic and let Sentieon figure it out

* refactor: Remove Sentieon license server IP check

* fix: Add a -n

* Try accessing secrets in the module?

* fix: Use Nextflow for if statement

* update snapshot

* update snapshot

* extend logic to all main.nf

* ci: Add Sentieon logic to pytest-workflow

* test(sentieon): Add env section to every config

* Revert "ci: Add Sentieon logic to pytest-workflow"

This reverts commit 76884d2.

* test: Remove pytest_modules for sentieon

---------

Co-authored-by: maxulysse <[email protected]>
…f-core#5978)

* ltrharvest: Added test to verify graceful handling of no ltr input

* Updated test data paths

* Fixed test data path

* Removed unstable items from snapshot

* Updated snapshotting logic
nf-core#5980)

* ltrretriever/ltrretriever: Added test to verify graceful handling of no ltr input

* Removed unstable items from snapshot
* Updated mkfastq module

* Combined input channels

* Updated main.nf.test and snap

---------

Co-authored-by: zxBIB Schcolnicov <[email protected]>
Co-authored-by: Anabella Trigila <[email protected]>
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.