Make code PEP8 compliant

Currently we only enforce PEP8 standards on lines of code that have changed in a PR. We need all code to be PEP8 compliant so we don't have a mixed message on our standards.

• minimally synapseclient and synapseutils should be free from PEP8 issues

• unit/integration tests free from PEP8 would be nice also if but not critical

• we should configure pep8speaks to use flake8 as its linter that way you can get the same lint result from the command line and in builds that you'd see after submitting a PR