Python pylint pep8

[benlast]

Python coding standards

Status: Opened for review

Reviewers: Any and all Python devs
JIRA: https://mobify.atlassian.net/browse/CRTM-82

Changes

• Added a big README to cover Python standards and code checking
• Added a standardized pylintrc

How to test-drive this PR

• Read the README
• Try pep8, pylint and pyflakes

[kyoung]

Run?

[benlast]

It's in the default... I believe it comes from some library package that uses it as a standard name (possibly in threading).

[roadsideseb]

From how I understand it, this is a setup and docs for running PEP8 checks but we don't run them automatically as part of the test script? I think it would be worth adding that into circle.yml to ensure it actually gets run by the developers.

[benlast]

@elbaschid I agree that running the checks automatically would be good, but that's a whole other rabbithole to go down, since we'd need agreement on whether tests fail if there are violations (which would mean portal became undeployable), or if there are new violations (how do we track old and find new), and so on.

[dbader]

Thanks for putting this together @benlast! This will give us enough of a foundation to try these tools and figure out where we need to do some more tuning. Btw this is an interesting article that talks about applying this in practice: http://www.jeffknupp.com/blog/2013/11/15/supercharge-your-python-developers/