search

beetbox / pyacoustid

Python bindings for Chromaprint acoustic fingerprinting and the Acoustid Web service
MIT License
325 stars 66 forks source link

Fix flake8 issues #40

Closed sumpfralle closed 6 years ago

sumpfralle commented 6 years ago

The following small flake8 (style) issues are fixed:

Now the module is flake8 clean (for python2 and python3).

sampsyo commented 6 years ago

Cool; thanks! We should really turn on flake8 checking in the CI.

I certainly agree with the spacing for the comments, but the class name is a bit of a special case. It's a class, yes, but it's also a decorator—and decorators are typically spelled with snake_case. Can we leave that one how it is?

sumpfralle commented 6 years ago

Thank you for your quick response!

Can we leave that one how it is?

Who am I to argue? :)

Now I used an override there instead of the CamelCase. Additionally I went through the other files and fixed their minor flake8 issues.

sampsyo commented 6 years ago

Looks awesome. Thanks again!