test/static-code: drop flake8

`ruff` in preview mode now implements the various E[123] rules that we
were depending on flake8 for.  Drop it.
This commit is contained in:
Allison Karlitskaya 2024-04-15 20:13:48 +02:00 committed by Jelle van der Waa
parent 5b6c1ce8ae
commit 6298dd93c8
4 changed files with 2 additions and 17 deletions

View File

@ -1,8 +0,0 @@
[flake8]
# Use flake8 only to cover rules not yet supported (or enabled by
# default) with ruff. See https://github.com/astral-sh/ruff/issues/2402
# and https://github.com/astral-sh/ruff/releases/tag/v0.0.269
select=
E1
E2
E3

View File

@ -525,7 +525,7 @@ For running tests, the following dependencies are required:
sudo dnf install curl expect xz rpm-build chromium-headless dbus-daemon \
libvirt-daemon-driver-storage-core libvirt-daemon-driver-qemu libvirt-client python3-libvirt \
python3-flake8 python3-pyyaml
python3-pyyaml
For compiling the C parts, you will need the package build dependencies:

View File

@ -142,7 +142,6 @@ wheel_build_env = venv-pkg
skip_install = lint: True
deps =
lint: mypy
lint: flake8
lint: ruff
lint: vulture
pytest

View File

@ -1,9 +1,8 @@
#!/bin/bash
# run static code checks like eslint, flake8, mypy, ruff, vulture.
# run static code checks like eslint, mypy, ruff, vulture.
set -eu
# requires: .flake8
# requires: pyproject.toml
# requires: containers/flatpak/test/ruff.toml
# requires: pkg/ruff.toml
@ -34,11 +33,6 @@ find_python_files() {
find_scripts 'python3' '*.py'
}
test_flake8() {
command -v flake8 >/dev/null || skip 'no flake8'
find_python_files | xargs -r -0 flake8
}
test_ruff() {
command -v ruff >/dev/null || skip 'no ruff'
find_python_files | xargs -r -0 ruff check --quiet --no-cache