summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJustin Mayer <entroP@gmail.com>2020-11-20 16:22:39 +0100
committerJustin Mayer <entroP@gmail.com>2020-11-20 16:22:39 +0100
commitf21969a0161621642f60139b5735190ee633e7e6 (patch)
treedd3b4fbdc10502fdc06bde6dc29730c3597af8b9
parent4b79d6dae67844ba7faff10101dc8b18780692d4 (diff)
downloadpelican-f21969a0161621642f60139b5735190ee633e7e6.tar.gz
Ensure `invoke lint` task & its docs are in sync
Someday the entire Pelican code base will perhaps have Black and isort formatting applied to it, but it doesn't make sense for the linter to check that until it's actually done. This changes the `lint` Invoke task to run Flake8 on changed lines only, which should bring it into line with both the current Pre-commit and CI configurations. Also, the docs erroneously stated that `invoke tests` would also check for code style compliance, which has herein been fixed.
-rw-r--r--docs/contribute.rst13
-rw-r--r--tasks.py4
2 files changed, 9 insertions, 8 deletions
diff --git a/docs/contribute.rst b/docs/contribute.rst
index 62ed51f9..2e1dc317 100644
--- a/docs/contribute.rst
+++ b/docs/contribute.rst
@@ -75,11 +75,14 @@ via::
invoke tests
-In addition to running the test suite, the above invocation will also check code
-style and let you know whether non-conforming patterns were found. In some cases
-these linters will make the needed changes directly, while in other cases you
-may need to make additional changes until ``invoke tests`` no longer reports any
-code style violations.
+In addition to running the test suite, it is important to also ensure that any
+lines you changed conform to code style guidelines. You can check that via::
+
+ invoke lint
+
+If code style violations are found in lines you changed, correct those lines
+and re-run the above lint command until they have all been fixed. You do not
+need to address style violations, if any, for code lines you did not touch.
After making your changes and running the tests, you may see a test failure
mentioning that "some generated files differ from the expected functional tests
diff --git a/tasks.py b/tasks.py
index 171a9711..1d511859 100644
--- a/tasks.py
+++ b/tasks.py
@@ -70,13 +70,11 @@ def isort(c, check=False, diff=False):
@task
def flake8(c):
- c.run(f"{VENV_BIN}/flake8 {PKG_PATH} tasks.py")
+ c.run(f"git diff HEAD | {VENV_BIN}/flake8 --diff --max-line-length=88")
@task
def lint(c):
- isort(c, check=True)
- black(c, check=True)
flake8(c)