qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 0/6] Python: delint python library


From: John Snow
Subject: Re: [PATCH v2 0/6] Python: delint python library
Date: Tue, 18 May 2021 09:33:15 -0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1

On 5/17/21 2:48 PM, John Snow wrote:
gitlab CI: https://gitlab.com/jsnow/qemu/-/pipelines/304224309
branch: https://gitlab.com/jsnow/qemu/-/commits/python-package-pre-cleanup

This series serves as a pre-requisite for packaging the python series
and getting the linters running via CI. The first patch fixes a linter
error we've had for a while now; the subsequent ones fix a new warning
that was recently added to pylint 2.8.x.

V2:
  - iotest bits already handled, dropped here.
  - Changed patch #3 based on feedback from Wainer.

John Snow (6):
   python/console_socket: avoid one-letter variable
   python/machine: use subprocess.DEVNULL instead of
     open(os.path.devnull)
   python/machine: use subprocess.run instead of subprocess.Popen
   python/console_socket: Add a pylint ignore
   python/machine: Disable pylint warning for open() in _pre_launch
   python/machine: disable warning for Popen in _launch()

  python/qemu/console_socket.py | 11 ++++++-----
  python/qemu/machine.py        | 28 ++++++++++++++++++----------
  2 files changed, 24 insertions(+), 15 deletions(-)


Thank you for reviews! Staged to my Python branch.

--js




reply via email to

[Prev in Thread] Current Thread [Next in Thread]