CI should implement static analysis and block merging for issues that arise

[pjrobertson]

Suggested to include:

1. pylint
2. flake8

These may want to be done at two separate stages, since pylint may only raise a few small issues, but flake8 will likely raise more (stylistic changes are likely to be more needed)

A quick test shows:

1. pylint: approx 1450 lines changed, Your code has been rated at 3.39/10 ( 😢 )
2. flake8: approx 1100 lines changed

We should also decide on a convention for code as necessary.