diff options
author | Justin Mayer <entroP@gmail.com> | 2020-11-20 16:22:39 +0100 |
---|---|---|
committer | Justin Mayer <entroP@gmail.com> | 2020-11-20 16:22:39 +0100 |
commit | f21969a0161621642f60139b5735190ee633e7e6 (patch) | |
tree | dd3b4fbdc10502fdc06bde6dc29730c3597af8b9 | |
parent | 4b79d6dae67844ba7faff10101dc8b18780692d4 (diff) | |
download | pelican-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.rst | 13 | ||||
-rw-r--r-- | tasks.py | 4 |
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 @@ -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) |