search

eastgenomics / eggd_dias_batch

1 stars 1 forks source link

v3.1.0 - add running mode to reports folder output name #188

Closed jethror1 closed 9 months ago

jethror1 commented 9 months ago

adds in running mode for SNV and mosaic to the reports workflow output directory name to fix #186

for CNVs this is omitted since the reports workflow name is different and contains CNV so therefore would be redundant.

Example new directory output naming would be:

image

This change is Reviewable

pep8speaks commented 9 months ago

Hello @jethror1! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 1000:80: E501 line too long (83 > 79 characters)

Line 544:5: E303 too many blank lines (2) Line 544:80: E501 line too long (87 > 79 characters)