[PATCH v5 0/3] [DO-NOT-MERGE] qapi: static typing conversion, "pt0"

John Snow posted 3 patches 7 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231004230532.3002201-1-jsnow@redhat.com
Maintainers: Peter Maydell <peter.maydell@linaro.org>, Markus Armbruster <armbru@redhat.com>, Michael Roth <michael.roth@amd.com>
docs/conf.py                          |  3 ++-
docs/devel/index.rst                  |  1 +
docs/devel/python/index.rst           |  7 +++++++
docs/devel/python/qapi.commands.rst   |  7 +++++++
docs/devel/python/qapi.common.rst     |  7 +++++++
docs/devel/python/qapi.error.rst      |  7 +++++++
docs/devel/python/qapi.events.rst     |  7 +++++++
docs/devel/python/qapi.expr.rst       |  7 +++++++
docs/devel/python/qapi.gen.rst        |  7 +++++++
docs/devel/python/qapi.introspect.rst |  7 +++++++
docs/devel/python/qapi.main.rst       |  7 +++++++
docs/devel/python/qapi.parser.rst     |  8 ++++++++
docs/devel/python/qapi.rst            | 26 ++++++++++++++++++++++++
docs/devel/python/qapi.schema.rst     |  7 +++++++
docs/devel/python/qapi.source.rst     |  7 +++++++
docs/devel/python/qapi.types.rst      |  7 +++++++
docs/devel/python/qapi.visit.rst      |  7 +++++++
scripts/qapi-lint.sh                  | 29 +++++++++++++++++++++++++++
scripts/qapi/Makefile                 |  5 +++++
scripts/qapi/gen.py                   |  2 +-
scripts/qapi/parser.py                |  5 +++--
21 files changed, 166 insertions(+), 4 deletions(-)
create mode 100644 docs/devel/python/index.rst
create mode 100644 docs/devel/python/qapi.commands.rst
create mode 100644 docs/devel/python/qapi.common.rst
create mode 100644 docs/devel/python/qapi.error.rst
create mode 100644 docs/devel/python/qapi.events.rst
create mode 100644 docs/devel/python/qapi.expr.rst
create mode 100644 docs/devel/python/qapi.gen.rst
create mode 100644 docs/devel/python/qapi.introspect.rst
create mode 100644 docs/devel/python/qapi.main.rst
create mode 100644 docs/devel/python/qapi.parser.rst
create mode 100644 docs/devel/python/qapi.rst
create mode 100644 docs/devel/python/qapi.schema.rst
create mode 100644 docs/devel/python/qapi.source.rst
create mode 100644 docs/devel/python/qapi.types.rst
create mode 100644 docs/devel/python/qapi.visit.rst
create mode 100755 scripts/qapi-lint.sh
create mode 100644 scripts/qapi/Makefile
[PATCH v5 0/3] [DO-NOT-MERGE] qapi: static typing conversion, "pt0"
Posted by John Snow 7 months ago
I'm sending this one out for Victor Toso's benefit this time. If you
aren't Victor Toso, you can probably mark this as read now. If you're
Markus, you can just assume everything in here is perfect and good.

This is a manual testing pre-requisite for other QAPI cleanup
patches. It isn't meant for inclusion, it's included here as an
informative basis for ongoing work only.

(Except the last patch, which is perfect and good, and Markus should
apply it without even looking at it. That's just how normal and good it
is.)

sphinx-autodoc is enabled for the scripts/qapi package so that docstring
formatting can be validated and inspected to make sure the output looks
"nice". It is not necessarily how we'd want to expose this information,
ultimately, but serves as a good smoke test for base-line docstring
consistency.

Lastly, some scripts/tools for running linters with "canonical"
configurations are provided as a means to verify the linting
base-line. I use these to validate my own work.

V5:
 - Rebased on today's origin/master;
 - Minor fix to re-establish baseline.

John Snow (3):
  [DO-NOT-MERGE] docs: enable sphinx-autodoc for scripts/qapi
  [DO-NOT-MERGE]: Add some ad-hoc linting helpers.
  qapi: re-establish linting baseline

 docs/conf.py                          |  3 ++-
 docs/devel/index.rst                  |  1 +
 docs/devel/python/index.rst           |  7 +++++++
 docs/devel/python/qapi.commands.rst   |  7 +++++++
 docs/devel/python/qapi.common.rst     |  7 +++++++
 docs/devel/python/qapi.error.rst      |  7 +++++++
 docs/devel/python/qapi.events.rst     |  7 +++++++
 docs/devel/python/qapi.expr.rst       |  7 +++++++
 docs/devel/python/qapi.gen.rst        |  7 +++++++
 docs/devel/python/qapi.introspect.rst |  7 +++++++
 docs/devel/python/qapi.main.rst       |  7 +++++++
 docs/devel/python/qapi.parser.rst     |  8 ++++++++
 docs/devel/python/qapi.rst            | 26 ++++++++++++++++++++++++
 docs/devel/python/qapi.schema.rst     |  7 +++++++
 docs/devel/python/qapi.source.rst     |  7 +++++++
 docs/devel/python/qapi.types.rst      |  7 +++++++
 docs/devel/python/qapi.visit.rst      |  7 +++++++
 scripts/qapi-lint.sh                  | 29 +++++++++++++++++++++++++++
 scripts/qapi/Makefile                 |  5 +++++
 scripts/qapi/gen.py                   |  2 +-
 scripts/qapi/parser.py                |  5 +++--
 21 files changed, 166 insertions(+), 4 deletions(-)
 create mode 100644 docs/devel/python/index.rst
 create mode 100644 docs/devel/python/qapi.commands.rst
 create mode 100644 docs/devel/python/qapi.common.rst
 create mode 100644 docs/devel/python/qapi.error.rst
 create mode 100644 docs/devel/python/qapi.events.rst
 create mode 100644 docs/devel/python/qapi.expr.rst
 create mode 100644 docs/devel/python/qapi.gen.rst
 create mode 100644 docs/devel/python/qapi.introspect.rst
 create mode 100644 docs/devel/python/qapi.main.rst
 create mode 100644 docs/devel/python/qapi.parser.rst
 create mode 100644 docs/devel/python/qapi.rst
 create mode 100644 docs/devel/python/qapi.schema.rst
 create mode 100644 docs/devel/python/qapi.source.rst
 create mode 100644 docs/devel/python/qapi.types.rst
 create mode 100644 docs/devel/python/qapi.visit.rst
 create mode 100755 scripts/qapi-lint.sh
 create mode 100644 scripts/qapi/Makefile

-- 
2.41.0