[PATCH 0/4] python: pylint and flake8 support

John Snow posted 4 patches 3 years, 11 months ago
Test docker-mingw@fedora passed
Test checkpatch passed
Test asan passed
Test docker-quick@centos7 passed
Test FreeBSD passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200528222129.23826-1-jsnow@redhat.com
Maintainers: Markus Armbruster <armbru@redhat.com>
python/qemu/.flake8    |  2 ++
python/qemu/accel.py   |  9 ++++---
python/qemu/machine.py | 52 +++++++++++++++++++++++--------------
python/qemu/pylintrc   | 58 ++++++++++++++++++++++++++++++++++++++++++
python/qemu/qmp.py     |  4 +--
python/qemu/qtest.py   | 42 +++++++++++++++++++-----------
scripts/qmp/qmp        |  4 ++-
scripts/qmp/qom-fuse   |  4 ++-
scripts/qmp/qom-get    |  6 +++--
scripts/qmp/qom-list   |  6 +++--
scripts/qmp/qom-set    |  6 +++--
scripts/qmp/qom-tree   |  6 +++--
12 files changed, 150 insertions(+), 49 deletions(-)
create mode 100644 python/qemu/.flake8
create mode 100644 python/qemu/pylintrc
[PATCH 0/4] python: pylint and flake8 support
Posted by John Snow 3 years, 11 months ago
This is a quick series to delint the files under python/qemu, with one
extra fix outside of that domain.

This was split out from my longer series attempting to package
python/qemu. This part is a nice standalone chunk.

John Snow (4):
  scripts/qmp: Fix shebang and imports
  python/machine.py: remove bare except
  python/qemu: delint and add pylintrc
  python/qemu: delint; add flake8 config

 python/qemu/.flake8    |  2 ++
 python/qemu/accel.py   |  9 ++++---
 python/qemu/machine.py | 52 +++++++++++++++++++++++--------------
 python/qemu/pylintrc   | 58 ++++++++++++++++++++++++++++++++++++++++++
 python/qemu/qmp.py     |  4 +--
 python/qemu/qtest.py   | 42 +++++++++++++++++++-----------
 scripts/qmp/qmp        |  4 ++-
 scripts/qmp/qom-fuse   |  4 ++-
 scripts/qmp/qom-get    |  6 +++--
 scripts/qmp/qom-list   |  6 +++--
 scripts/qmp/qom-set    |  6 +++--
 scripts/qmp/qom-tree   |  6 +++--
 12 files changed, 150 insertions(+), 49 deletions(-)
 create mode 100644 python/qemu/.flake8
 create mode 100644 python/qemu/pylintrc

-- 
2.21.3


Re: [PATCH 0/4] python: pylint and flake8 support
Posted by Philippe Mathieu-Daudé 3 years, 11 months ago
On 5/29/20 12:21 AM, John Snow wrote:
> This is a quick series to delint the files under python/qemu, with one
> extra fix outside of that domain.
> 
> This was split out from my longer series attempting to package
> python/qemu. This part is a nice standalone chunk.
> 
> John Snow (4):
>   scripts/qmp: Fix shebang and imports
>   python/machine.py: remove bare except
>   python/qemu: delint and add pylintrc
>   python/qemu: delint; add flake8 config
> 
>  python/qemu/.flake8    |  2 ++
>  python/qemu/accel.py   |  9 ++++---
>  python/qemu/machine.py | 52 +++++++++++++++++++++++--------------
>  python/qemu/pylintrc   | 58 ++++++++++++++++++++++++++++++++++++++++++
>  python/qemu/qmp.py     |  4 +--
>  python/qemu/qtest.py   | 42 +++++++++++++++++++-----------
>  scripts/qmp/qmp        |  4 ++-
>  scripts/qmp/qom-fuse   |  4 ++-
>  scripts/qmp/qom-get    |  6 +++--
>  scripts/qmp/qom-list   |  6 +++--
>  scripts/qmp/qom-set    |  6 +++--
>  scripts/qmp/qom-tree   |  6 +++--
>  12 files changed, 150 insertions(+), 49 deletions(-)
>  create mode 100644 python/qemu/.flake8
>  create mode 100644 python/qemu/pylintrc
> 

Thanks, patches 1/3/4 applied to my python-next tree:
https://gitlab.com/philmd/qemu/commits/python-next

I skipped patch #2 which doesn't apply on top on Vladimir's "hard-kill"
patch:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg680200.html