search

sybrenstuvel / flickrapi

Python Flickr API implementation
https://stuvel.eu/flickrapi
Other
155 stars 33 forks source link

Add support for Python 3.7 and fix Flake8 #119

Closed hugovk closed 4 years ago

hugovk commented 5 years ago

Add support for Python 3.7, it needs Xenial on Travis CI.

(Also sudo: is no longer required: https://blog.travis-ci.com/2018-11-19-required-linux-infrastructure-migration)

A new release of Flake8 has caused the QA build job to fail with:

W504 line break after binary operator

Fixing these cause: 

W503 line break before binary operator

W503 and W504 are not compatible with each other, so one needs ignoring. W503 is not compatible with PEP 8, so ignore it. (If you prefer it the other way, let's ignore the other one.)

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 71.156% when pulling 3d491e8131f3b8a59c9c735726bff9082552aee3 on hugovk:update-ci into a4a9670dfbafcd0fad8044c363e5ff1780269253 on sybrenstuvel:master.

sybrenstuvel commented 4 years ago

Thanks! I've cherrypicked two of the four commits from this PR, namely the Flake8 ones. I'm migrating the package management to Poetry and will add Python 3.7 support there too.