search

hasl-sensor / integration

Swedish Public Transport Sensor (HASL). Formerly named HomeAssistant SL Sensor
https://hasl.sorlov.com
Apache License 2.0
34 stars 7 forks source link

New data format with more detailed info #25

Closed clearminds closed 2 years ago

pep8speaks commented 2 years ago

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

Line 223:80: E501 line too long (111 > 79 characters) Line 243:80: E501 line too long (86 > 79 characters) Line 245:80: E501 line too long (105 > 79 characters) Line 246:80: E501 line too long (91 > 79 characters) Line 247:80: E501 line too long (118 > 79 characters) Line 254:80: E501 line too long (99 > 79 characters) Line 255:80: E501 line too long (82 > 79 characters) Line 256:80: E501 line too long (89 > 79 characters) Line 258:80: E501 line too long (120 > 79 characters) Line 259:80: E501 line too long (101 > 79 characters) Line 260:80: E501 line too long (133 > 79 characters) Line 263:80: E501 line too long (86 > 79 characters) Line 264:80: E501 line too long (86 > 79 characters) Line 265:80: E501 line too long (84 > 79 characters) Line 266:80: E501 line too long (84 > 79 characters) Line 267:80: E501 line too long (201 > 79 characters) Line 268:80: E501 line too long (99 > 79 characters) Line 275:80: E501 line too long (82 > 79 characters) Line 276:80: E501 line too long (82 > 79 characters) Line 279:80: E501 line too long (87 > 79 characters) Line 280:80: E501 line too long (106 > 79 characters) Line 284:80: E501 line too long (87 > 79 characters) Line 285:80: E501 line too long (104 > 79 characters) Line 299:80: E501 line too long (92 > 79 characters) Line 300:80: E501 line too long (83 > 79 characters) Line 304:80: E501 line too long (84 > 79 characters) Line 307:80: E501 line too long (95 > 79 characters) Line 310:80: E501 line too long (97 > 79 characters) Line 319:38: E111 indentation is not a multiple of four Line 319:38: E117 over-indented Line 321:38: E111 indentation is not a multiple of four Line 321:38: E117 over-indented Line 323:38: E111 indentation is not a multiple of four Line 323:38: E117 over-indented Line 325:33: E265 block comment should start with '# ' Line 334:80: E501 line too long (85 > 79 characters) Line 344:80: E501 line too long (95 > 79 characters) Line 374:80: E501 line too long (80 > 79 characters) Line 378:80: E501 line too long (90 > 79 characters) Line 381:80: E501 line too long (127 > 79 characters) Line 498:80: E501 line too long (86 > 79 characters) Line 533:80: E501 line too long (86 > 79 characters) Line 628:80: E501 line too long (81 > 79 characters)

Comment last updated at 2022-01-18 21:08:31 UTC
DSorlov commented 2 years ago

@clearminds this commit failed validation so It cannot be merged.