mbox

[PULL,00/13] QAPI patches patches for 2021-10-02

Message ID 20211002095655.1944970-1-armbru@redhat.com
State New
Headers show

Pull-request

git://repo.or.cz/qemu/armbru.git tags/pull-qapi-2021-10-02

Message

Markus Armbruster Oct. 2, 2021, 9:56 a.m. UTC
The following changes since commit 5f992102383ed8ed97076548e1c897c7034ed8a4:

  Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2021-10-01 13:44:36 -0400)

are available in the Git repository at:

  git://repo.or.cz/qemu/armbru.git tags/pull-qapi-2021-10-02

for you to fetch changes up to d183e0481b1510b253ac94e702c76115f3bb6450:

  qapi/parser: enable pylint checks (2021-10-02 07:33:42 +0200)

----------------------------------------------------------------
QAPI patches patches for 2021-10-02

----------------------------------------------------------------
John Snow (13):
      qapi/pylintrc: ignore 'consider-using-f-string' warning
      qapi/gen: use dict.items() to iterate over _modules
      qapi/parser: fix unused check_args_section arguments
      qapi: Add spaces after symbol declaration for consistency
      qapi/parser: remove FIXME comment from _append_body_line
      qapi/parser: clarify _end_section() logic
      qapi/parser: Introduce NullSection
      qapi/parser: add import cycle workaround
      qapi/parser: add type hint annotations (QAPIDoc)
      qapi/parser: Add FIXME for consolidating JSON-related types
      qapi/parser: enable mypy checks
      qapi/parser: Silence too-few-public-methods warning
      qapi/parser: enable pylint checks

 qapi/block-core.json                   |   1 +
 qga/qapi-schema.json                   |   3 +
 scripts/qapi/gen.py                    |   3 +-
 scripts/qapi/mypy.ini                  |   5 --
 scripts/qapi/parser.py                 | 148 +++++++++++++++++++++------------
 scripts/qapi/pylintrc                  |   4 +-
 tests/qapi-schema/doc-bad-feature.err  |   2 +-
 tests/qapi-schema/doc-empty-symbol.err |   2 +-
 tests/qapi-schema/doc-good.json        |   8 ++
 9 files changed, 112 insertions(+), 64 deletions(-)

Comments

Richard Henderson Oct. 2, 2021, 8:31 p.m. UTC | #1
On 10/2/21 5:56 AM, Markus Armbruster wrote:
> The following changes since commit 5f992102383ed8ed97076548e1c897c7034ed8a4:
> 
>    Merge remote-tracking branch 'remotes/bonzini/tags/for-upstream' into staging (2021-10-01 13:44:36 -0400)
> 
> are available in the Git repository at:
> 
>    git://repo.or.cz/qemu/armbru.git tags/pull-qapi-2021-10-02
> 
> for you to fetch changes up to d183e0481b1510b253ac94e702c76115f3bb6450:
> 
>    qapi/parser: enable pylint checks (2021-10-02 07:33:42 +0200)
> 
> ----------------------------------------------------------------
> QAPI patches patches for 2021-10-02
> 
> ----------------------------------------------------------------
> John Snow (13):
>        qapi/pylintrc: ignore 'consider-using-f-string' warning
>        qapi/gen: use dict.items() to iterate over _modules
>        qapi/parser: fix unused check_args_section arguments
>        qapi: Add spaces after symbol declaration for consistency
>        qapi/parser: remove FIXME comment from _append_body_line
>        qapi/parser: clarify _end_section() logic
>        qapi/parser: Introduce NullSection
>        qapi/parser: add import cycle workaround
>        qapi/parser: add type hint annotations (QAPIDoc)
>        qapi/parser: Add FIXME for consolidating JSON-related types
>        qapi/parser: enable mypy checks
>        qapi/parser: Silence too-few-public-methods warning
>        qapi/parser: enable pylint checks

Thanks, applied.

r~