search

eastgenomics / eggd_generate_variant_workbook

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

IN-501 Add a report text #178

Closed Aisha-D closed 3 months ago

Aisha-D commented 3 months ago

Makes a report_text column similiar to ones in the uranus workbooks.

This change is Reviewable

pep8speaks commented 3 months ago

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

Line 286:13: E124 closing bracket does not match visual indentation

Line 1045:26: E203 whitespace before ':' Line 1129:1: W391 blank line at end of file

Line 356:1: W293 blank line contains whitespace Line 359:37: E225 missing whitespace around operator Line 375:80: E501 line too long (81 > 79 characters) Line 474:5: E303 too many blank lines (2) Line 686:5: E303 too many blank lines (2) Line 714:1: W293 blank line contains whitespace Line 950:13: E122 continuation line missing indentation or outdented Line 952:80: E501 line too long (89 > 79 characters) Line 955:80: E501 line too long (98 > 79 characters) Line 956:80: E501 line too long (103 > 79 characters) Line 957:80: E501 line too long (98 > 79 characters) Line 959:13: E122 continuation line missing indentation or outdented Line 960:9: E122 continuation line missing indentation or outdented

Comment last updated at 2024-03-18 16:45:51 UTC