Add flake8 config and fix linting warnings - Githubissues

JamesRamm / archook

Searches the system for arcgis and makes arcpy available to python (regardless of pythonpath/system path/registry settings)

GNU General Public License v2.0

81 stars 28 forks source link

Add flake8 config and fix linting warnings #19

Closed logan-pugh closed 4 years ago

logan-pugh commented 4 years ago

This PR adds a config for the flake8 linter and fixes all the warnings and errors it indicated.

Also standardizes on single-quotes since it previously was using a mix of single and double quotes, and tweaks some whitespace/indentation.

The ignored rules in the .flake8 config file are as follows:

E501: Line too long
W504: Line break occurred after a binary operator
E261: At least two spaces before inline comment

Tip: If you use VS Code you can select flake8 from the Python: Select Linter command.

mhw-at-yg commented 4 years ago

+1 I've a growing liking for Black and might add something like https://ljvmiranda921.github.io/notebook/2018/06/21/precommits-using-black-and-flake8/ (though automation on a project this small is overkill ;-)