Closed Yu-jinKim closed 4 months ago
Hello @Yu-jinKim! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
trendyqc/trend_monitoring/models/fastq_qc.py
:Line 7:80: E501 line too long (141 > 79 characters) Line 8:80: E501 line too long (141 > 79 characters)
trendyqc/trend_monitoring/tests/test_multiqc.py
:Line 39:80: E501 line too long (88 > 79 characters) Line 51:80: E501 line too long (82 > 79 characters) Line 62:80: E501 line too long (83 > 79 characters) Line 103:80: E501 line too long (84 > 79 characters) Line 131:39: W291 trailing whitespace Line 147:12: W291 trailing whitespace Line 149:80: E501 line too long (93 > 79 characters) Line 177:80: E501 line too long (93 > 79 characters) Line 201:35: W291 trailing whitespace Line 293:80: E501 line too long (83 > 79 characters) Line 300:80: E501 line too long (101 > 79 characters) Line 329:80: E501 line too long (88 > 79 characters)
List of tests:
This change is![Reviewable](https://reviewable.io/review_button.svg)