eastgenomics / trendyQC

Django app for monitoring trends in MultiQC data
MIT License
0 stars 0 forks source link

In 636 unittesting plotting related functions create trace #100

Closed Yu-jinKim closed 1 week ago

Yu-jinKim commented 1 week ago

This change is Reviewable

pep8speaks commented 1 week ago

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

Line 253:80: E501 line too long (179 > 79 characters) Line 255:80: E501 line too long (179 > 79 characters) Line 256:80: E501 line too long (179 > 79 characters) Line 257:80: E501 line too long (179 > 79 characters) Line 258:80: E501 line too long (179 > 79 characters)

Line 498:80: E501 line too long (80 > 79 characters) Line 499:80: E501 line too long (80 > 79 characters) Line 500:80: E501 line too long (80 > 79 characters) Line 564:80: E501 line too long (81 > 79 characters) Line 565:80: E501 line too long (109 > 79 characters) Line 567:80: E501 line too long (89 > 79 characters) Line 632:80: E501 line too long (85 > 79 characters) Line 633:80: E501 line too long (95 > 79 characters) Line 634:80: E501 line too long (128 > 79 characters) Line 636:80: E501 line too long (103 > 79 characters) Line 637:80: E501 line too long (96 > 79 characters) Line 638:80: E501 line too long (96 > 79 characters) Line 878:80: E501 line too long (90 > 79 characters)