[PATCH v4 3/4] qapi: Update flake8 config

John Snow posted 4 patches 3 years ago
Maintainers: Peter Maydell <peter.maydell@linaro.org>, Markus Armbruster <armbru@redhat.com>, Michael Roth <michael.roth@amd.com>
There is a newer version of this series
[PATCH v4 3/4] qapi: Update flake8 config
Posted by John Snow 3 years ago
New versions of flake8 don't like same-line comments. (It's a version
newer than what fc37 ships, but it still makes my life easier to fix it
now.)

Signed-off-by: John Snow <jsnow@redhat.com>
---
 scripts/qapi/.flake8 | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/qapi/.flake8 b/scripts/qapi/.flake8
index 6b158c68b84..a873ff67309 100644
--- a/scripts/qapi/.flake8
+++ b/scripts/qapi/.flake8
@@ -1,2 +1,3 @@
 [flake8]
-extend-ignore = E722  # Prefer pylint's bare-except checks to flake8's
+# Prefer pylint's bare-except checks to flake8's
+extend-ignore = E722
-- 
2.39.0
Re: [PATCH v4 3/4] qapi: Update flake8 config
Posted by Markus Armbruster 3 years ago
John Snow <jsnow@redhat.com> writes:

> New versions of flake8 don't like same-line comments. (It's a version
> newer than what fc37 ships, but it still makes my life easier to fix it
> now.)
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
>  scripts/qapi/.flake8 | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/qapi/.flake8 b/scripts/qapi/.flake8
> index 6b158c68b84..a873ff67309 100644
> --- a/scripts/qapi/.flake8
> +++ b/scripts/qapi/.flake8
> @@ -1,2 +1,3 @@
>  [flake8]
> -extend-ignore = E722  # Prefer pylint's bare-except checks to flake8's
> +# Prefer pylint's bare-except checks to flake8's
> +extend-ignore = E722

Reviewed-by: Markus Armbruster <armbru@redhat.com>