Issues from code review

eastgenomics / eggd_generate_variant_workbook

DNAnexus app for generating xlsx variant workbooks

3 stars 0 forks source link

Issues from code review #11

Closed Yu-jinKim closed 4 years ago

Yu-jinKim commented 4 years ago

This change is

pep8speaks commented 4 years ago

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

In the file resources/home/dnanexus/get_transcript_regions.py:

Line 29:80: E501 line too long (82 > 79 characters) Line 118:80: E501 line too long (89 > 79 characters) Line 119:80: E501 line too long (97 > 79 characters) Line 120:80: E501 line too long (87 > 79 characters)

mattgarner commented 4 years ago

These changes address issues identified here: https://github.com/eastgenomics/eggd_vcf2xls_nirvana/pull/9