Closed Yu-jinKim closed 5 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/tests/test_multiqc.py
:Line 44:1: E302 expected 2 blank lines, found 1 Line 92:80: E501 line too long (80 > 79 characters) Line 114:80: E501 line too long (89 > 79 characters) Line 136:35: W291 trailing whitespace Line 173:80: E501 line too long (82 > 79 characters) Line 300:80: E501 line too long (80 > 79 characters) Line 368:80: E501 line too long (81 > 79 characters) Line 425:80: E501 line too long (111 > 79 characters) Line 441:80: E501 line too long (82 > 79 characters) Line 494:80: E501 line too long (83 > 79 characters)
test_import_fastqc
andtest_import_picard
functionsThis change is