ubuntu-dev-tools/run-linters

#!/bin/sh
set -eu

# Copyright 2023, Canonical Ltd.
# SPDX-License-Identifier: GPL-3.0

PYTHON_SCRIPTS=$(grep -l -r '^#! */usr/bin/python3$' .)

echo "Running black..."
black --check --diff . $PYTHON_SCRIPTS

echo "Running isort..."
isort --check-only --diff .

echo "Running flake8..."
flake8 --max-line-length=99 --ignore=E203,W503 . $PYTHON_SCRIPTS

echo "Running pylint..."
pylint $(find * -name '*.py') $PYTHON_SCRIPTS
refactor: Move linter checks into run-linters script Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-30 14:10:02 +01:00			`#!/bin/sh`
			`set -eu`

			`# Copyright 2023, Canonical Ltd.`
			`# SPDX-License-Identifier: GPL-3.0`

fix: Check Python scripts with flake8 again Commit ae74f71a1e9d4be043162b19d23f2d44c964c771 removed the flake8 unittest and commit 3428a65b1cd644445f55ad8ae65ece5f73d7acb5 added running flake8 again, but only for files named `*.py`. Check also all Python scripts with a Python shebang. Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-30 19:29:51 +01:00			`PYTHON_SCRIPTS=$(grep -l -r '^#! */usr/bin/python3$' .)`

Run black code formatter during package build Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-30 19:48:49 +01:00			`echo "Running black..."`
			`black --check --diff . $PYTHON_SCRIPTS`

Run isort import sorter during package build Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-30 21:34:24 +01:00			`echo "Running isort..."`
			`isort --check-only --diff .`

refactor: Move linter checks into run-linters script Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-30 14:10:02 +01:00			`echo "Running flake8..."`
style: Format Python code with black ``` PYTHON_SCRIPTS=$(grep -l -r '^#! */usr/bin/python3$' .) black -C -l 99 . $PYTHON_SCRIPTS ``` Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-30 19:45:36 +01:00			`flake8 --max-line-length=99 --ignore=E203,W503 . $PYTHON_SCRIPTS`
Run pylint during package build again Commit ae74f71a1e9d4be043162b19d23f2d44c964c771 removed the pylint unit test saying that unit tests are not needed to just run flake8 or pylint. Since pylint is useful, add it back, but this time call it directly and not embed it into a unit test. Signed-off-by: Benjamin Drung <benjamin.drung@canonical.com> 2023-01-31 00:05:15 +01:00
			`echo "Running pylint..."`
			`pylint $(find * -name '*.py') $PYTHON_SCRIPTS`