Configure linting and add to pyinvoke

Author: gonzalocasas

CONTRIBUTING.rst

@@ -29,9 +29,15 @@ We love pull requests from everyone! Here's a quick guide to improve the code:
 
     invoke test
 
-7. Add yourself to ``AUTHORS.rst``.
-8. Commit your changes and push your branch to GitHub.
-9. Create a `pull request <https://help.github.com/articles/about-pull-requests/>`_ through the GitHub website.
+7. Check there are no linter errors:
+
+::
+
+    invoke lint
+
+8. Add yourself to ``AUTHORS.rst``.
+9. Commit your changes and push your branch to GitHub.
+10. Create a `pull request <https://help.github.com/articles/about-pull-requests/>`_ through the GitHub website.
 
 
 During development, use `pyinvoke <http://docs.pyinvoke.org/>`_ tasks on the
@@ -40,6 +46,7 @@ command prompt to ease recurring operations:
 * ``invoke clean``: Clean all generated artifacts.
 * ``invoke check``: Run various code and documentation style checks.
 * ``invoke docs``: Generate documentation.
+* ``invoke lint``: Run code linter for coding style checks.
 * ``invoke test``: Run all tests and checks in one swift command.
 * ``invoke``: Show available tasks.
 

setup.cfg

@@ -3,7 +3,15 @@ universal = 1
 
 [flake8]
 max-line-length = 180
-exclude = */migrations/*
+exclude =
+    .git,
+    __pycache__,
+    docs,
+    build,
+    temp,
+    dist,
+    src/compas_fab/backends/vrep/remote_api/*,
+    src/compas_fab/ghpython/path_planning.py
 
 [tool:pytest]
 testpaths = tests

tasks.py

@@ -152,18 +152,24 @@ def docs(ctx, doctest=False, rebuild=False, check_links=False):
         ctx.run('sphinx-build -b linkcheck docs dist/docs')
 
 
+@task()
+def lint(ctx):
+    """Check the consistency of coding style."""
+    log.write('Running flake8 python linter...')
+    ctx.run('flake8 src')
+
+
 @task()
 def check(ctx):
     """Check the consistency of documentation, coding style and a few other things."""
+    lint(ctx)
+
     log.write('Checking MANIFEST.in...')
     ctx.run('check-manifest --ignore-bad-ideas=remoteApi.so')
 
     log.write('Checking ReStructuredText formatting...')
     ctx.run('python setup.py check --strict --metadata --restructuredtext')
 
-    # log.write('Running flake8 python linter...')
-    # ctx.run('flake8 src setup.py')
-
     # log.write('Checking python imports...')
     # ctx.run('isort --check-only --diff --recursive src tests setup.py')
 
@@ -192,14 +198,10 @@ def deploy_docs(ctx):
 
 
 @task(help={
-      'checks': 'True to run all checks before testing, otherwise False.',
       'doctest': 'True to run doctest modules, otherwise False.',
       })
-def test(ctx, checks=False, doctest=True):
+def test(ctx, doctest=True):
     """Run all tests."""
-    if checks:
-        check(ctx)
-
     if doctest:
         ctx.run('pytest --doctest-modules')
     else: