mattjegan / django-gamification

The missing Django Gamification Package
Other
166 stars 28 forks source link

Flake8 Integration #29

Closed dhdavvie closed 6 years ago

dhdavvie commented 6 years ago

As mentioned in Issue #25 and #10, I have implemented flake8 into the Travis config and done some quick refactoring to satisfy the errors raised. To run the flake8 locally, simply call run $ flake8 in the root dir. I was not sure if you wanted flake8 integrated into the testing so that it is also run when you do coverage run manage.py tests. Also, some of the editings might need some looking at, notably the ones in test_models.py as that had some very long assert statements. All green on Travis for me. Please let me know if there is something you wanted doing differently!