John Snow <js...@redhat.com> writes: [...]
> diff --git a/python/tests/qapi-flake8.sh b/python/tests/qapi-flake8.sh > new file mode 100755 > index 00000000000..2211b6c7f41 > --- /dev/null > +++ b/python/tests/qapi-flake8.sh > @@ -0,0 +1,6 @@ > +#!/bin/sh -e > +# SPDX-License-Identifier: LGPL-2.1-or-later Why not GPL-2.0-or-later? > + > +python3 -m flake8 ../scripts/qapi/ \ > + ../docs/sphinx/qapidoc.py \ > + ../docs/sphinx/qapi_domain.py > diff --git a/python/tests/qapi-isort.sh b/python/tests/qapi-isort.sh > new file mode 100755 > index 00000000000..e546f5a7b5c > --- /dev/null > +++ b/python/tests/qapi-isort.sh > @@ -0,0 +1,8 @@ > +#!/bin/sh -e > +# SPDX-License-Identifier: LGPL-2.1-or-later Likewise. > + > +python3 -m isort --sp . -c ../scripts/qapi/ > +# Force isort to recognize "compat" as a local module and not third-party > +python3 -m isort --sp . -c -p compat -p qapidoc_legacy \ > + ../docs/sphinx/qapi_domain.py \ > + ../docs/sphinx/qapidoc.py > diff --git a/python/tests/qapi-mypy.sh b/python/tests/qapi-mypy.sh > new file mode 100755 > index 00000000000..50fb8ed1e3e > --- /dev/null > +++ b/python/tests/qapi-mypy.sh > @@ -0,0 +1,4 @@ > +#!/bin/sh -e > +# SPDX-License-Identifier: LGPL-2.1-or-later Likewise. > + > +python3 -m mypy ../scripts/qapi > diff --git a/python/tests/qapi-pylint.sh b/python/tests/qapi-pylint.sh > new file mode 100755 > index 00000000000..5f1b79c1883 > --- /dev/null > +++ b/python/tests/qapi-pylint.sh > @@ -0,0 +1,8 @@ > +#!/bin/sh -e > +# SPDX-License-Identifier: LGPL-2.1-or-later Likewise. > + > +SETUPTOOLS_USE_DISTUTILS=stdlib python3 -m pylint \ > + --rcfile=../scripts/qapi/pylintrc \ > + ../scripts/qapi/ \ > + ../docs/sphinx/qapidoc.py \ > + ../docs/sphinx/qapi_domain.py > diff --git a/scripts/qapi/pylintrc b/scripts/qapi/pylintrc > index d24eece7411..e16283ada3d 100644 > --- a/scripts/qapi/pylintrc > +++ b/scripts/qapi/pylintrc > @@ -19,6 +19,7 @@ disable=consider-using-f-string, > too-many-instance-attributes, > too-many-positional-arguments, > too-many-statements, > + unknown-option-value, > useless-option-value, > > [REPORTS]
