Add flake8 and clean up formatting

servo / highfive

Github hooks to provide an encouraging atmosphere for new contributors

Mozilla Public License 2.0

254 stars 58 forks source link

Closed aneeshusa closed 8 years ago

aneeshusa commented 8 years ago

Fix all the warnings from flake8, and add it to the Travis test suite.

highfive commented 8 years ago

Heads up! This PR modifies the following files:

@jdm: handlers/homu_status/init.py, test.py, handlers/assign_reviewer/init.py, handlers/empty_title_element/init.py, handlers/watchers/init.py, handlers/no_modify_css_tests/init.py, handlers/nonini_wpt_meta/init.py, helpers.py, handlers/unsafe/init.py, requirements.txt, handlers/missing_test/init.py, .travis.yml, newpr.py, eventhandler.py, handlers/status_update/init.py

highfive commented 8 years ago

warning Warning warning

These commits include an empty title element, ''. Consider adding appropriate metadata.
These commits modify unsafe code. Please review it carefully!

jdm commented 8 years ago

Thank you! This is great!