search

patchew-project / patchew

A patch email tracking and testing system
MIT License
72 stars 24 forks source link

Apply pep8 rules in api and www/views #95

Closed caiocarrara closed 5 years ago

caiocarrara commented 5 years ago

Most of the changes are simple indentation and blank lines spaces. There was also some import fixes that shuldn't change any behavior.

Probably the biggest change was in the method has_permission() of PatchewPermission class (api/rest.py). Firstly the use of backslash inside braces is redundant (to break lines). So n order to remove the backslashes and trying to make the permission conditions more readable I've tried the following approach.

If these changes be accepted I'm willing to continue applying it on other files.

caiocarrara commented 5 years ago

Checking the broking tests...