search

eastgenomics / eggd_generate_variant_workbook

DNAnexus app for generating xlsx variant workbooks
3 stars 0 forks source link

IN-510 unit test new features #192

Closed Aisha-D closed 2 months ago

Aisha-D commented 2 months ago

This change is Reviewable

pep8speaks commented 2 months ago

Hello @Aisha-D! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 212:41: E251 unexpected spaces around keyword / parameter equals Line 212:43: E251 unexpected spaces around keyword / parameter equals Line 636:73: E251 unexpected spaces around keyword / parameter equals Line 636:75: E251 unexpected spaces around keyword / parameter equals Line 691:1: E302 expected 2 blank lines, found 1 Line 699:80: E501 line too long (89 > 79 characters) Line 723:80: E501 line too long (89 > 79 characters) Line 736:27: E222 multiple spaces after operator Line 743:35: E231 missing whitespace after ',' Line 745:80: E501 line too long (102 > 79 characters)

Comment last updated at 2024-04-23 14:35:18 UTC