Raahul-Singh / pythia

Pythia, for Solar Active Region Data Analysis
https://raahul-singh.github.io/pythia/
MIT License
13 stars 2 forks source link

Adds SEO #45

Closed Raahul-Singh closed 4 years ago

Raahul-Singh commented 4 years ago

We have a working Search Events Object!

pep8speaks commented 4 years ago

Hello @Raahul-Singh! Thanks for updating this PR.

Line 1:38: E261 at least two spaces before inline comment

Line 774:101: E501 line too long (107 > 100 characters) Line 873:101: E501 line too long (111 > 100 characters) Line 903:101: E501 line too long (117 > 100 characters) Line 905:101: E501 line too long (118 > 100 characters) Line 906:101: E501 line too long (118 > 100 characters) Line 907:101: E501 line too long (118 > 100 characters) Line 908:101: E501 line too long (118 > 100 characters) Line 909:101: E501 line too long (118 > 100 characters) Line 933:45: E203 whitespace before ':'

Line 164:24: E203 whitespace before ':' Line 165:21: E203 whitespace before ':' Line 166:13: E123 closing bracket does not match indentation of opening bracket's line Line 170:101: E501 line too long (104 > 100 characters) Line 193:24: E203 whitespace before ':' Line 194:21: E203 whitespace before ':' Line 195:13: E123 closing bracket does not match indentation of opening bracket's line

Line 234:101: E501 line too long (104 > 100 characters) Line 235:101: E501 line too long (104 > 100 characters) Line 236:101: E501 line too long (104 > 100 characters) Line 237:101: E501 line too long (121 > 100 characters) Line 239:101: E501 line too long (104 > 100 characters) Line 240:101: E501 line too long (122 > 100 characters)

Line 81:52: E231 missing whitespace after ','

Comment last updated at 2020-08-25 10:24:26 UTC
Raahul-Singh commented 4 years ago

The codestyle test cannot be rectified, as it is only an alphabetical order shift of imports. But changing the order of imports causes other test fails.

dpshelio commented 4 years ago

The codestyle test cannot be rectified, as it is only an alphabetical order shift of imports. But changing the order of imports causes other test fails.

Why does the order of imports make things to fail?