qapi: misc testing and doc patches

[PATCH 01/10] qapi: update pylintrc config

Posted by John Snow 1 month, 1 week ago

If you've got a newer pylint, it'll whine about positional arguments
separately from the regular ones. Update the configuration to ignore
both categories of warning.

Signed-off-by: John Snow <jsnow@redhat.com>
---
 scripts/qapi/pylintrc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/qapi/pylintrc b/scripts/qapi/pylintrc
index c028a1f9f51..d24eece7411 100644
--- a/scripts/qapi/pylintrc
+++ b/scripts/qapi/pylintrc
@@ -17,6 +17,7 @@ disable=consider-using-f-string,
         too-many-arguments,
         too-many-branches,
         too-many-instance-attributes,
+        too-many-positional-arguments,
         too-many-statements,
         useless-option-value,
 
-- 
2.48.1

Re: [PATCH 01/10] qapi: update pylintrc config

Posted by Markus Armbruster 1 month ago

John Snow <jsnow@redhat.com> writes:

> If you've got a newer pylint, it'll whine about positional arguments
> separately from the regular ones. Update the configuration to ignore
> both categories of warning.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
>  scripts/qapi/pylintrc | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/scripts/qapi/pylintrc b/scripts/qapi/pylintrc
> index c028a1f9f51..d24eece7411 100644
> --- a/scripts/qapi/pylintrc
> +++ b/scripts/qapi/pylintrc
> @@ -17,6 +17,7 @@ disable=consider-using-f-string,
>          too-many-arguments,
>          too-many-branches,
>          too-many-instance-attributes,
> +        too-many-positional-arguments,
>          too-many-statements,
>          useless-option-value,

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