[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST |
Date: |
Thu, 20 Jun 2024 15:54:53 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
John Snow <jsnow@redhat.com> writes:
> We do not need a dedicated section for notes. By eliminating a specially
> parsed section, these notes can be treated as normal rST paragraphs in
> the new QMP reference manual, and can be placed and styled much more
> flexibly.
>
> Convert all existing "Note" and "Notes" sections to pure rST. As part of
> the conversion, capitalize the first letter of each sentence and add
> trailing punctuation where appropriate to ensure notes look sensible and
> consistent in rendered HTML documentation.
>
> Update docs/devel/qapi-code-gen.rst to reflect the new paradigm, and ...
>
> ... Update the QAPI parser to prohibit "Note" sections while suggesting
> a new syntax. The exact formatting to use is a matter of taste, but a
> good candidate is simply:
>
> .. note:: lorem ipsum ...
>
> ... but there are other choices, too. The Sphinx readthedocs theme
> offers theming for the following forms (capitalization unimportant); all
> are adorned with a (!) symbol in the title bar for rendered HTML docs.
>
> See
> https://sphinx-rtd-theme.readthedocs.io/en/stable/demo/demo.html#admonitions
> for examples of each directive/admonition in use.
>
> These are rendered in orange:
>
> .. Attention:: ...
> .. Caution:: ...
> .. WARNING:: ...
>
> These are rendered in red:
>
> .. DANGER:: ...
> .. Error:: ...
>
> These are rendered in green:
>
> .. Hint:: ...
> .. Important:: ...
> .. Tip:: ...
>
> These are rendered in blue:
>
> .. Note:: ...
> .. admonition:: custom title
>
> admonition body text
>
> This patch uses ".. note::" almost everywhere,
Not mentioned, and may or may not be worth mentioning: both "Note:" and
"Notes:" become ".. note::", which renders as "Note". One instance
quoted below.
No objection to the change; you obviously double-checked it reads okay
that way.
> with just two "caution"
> directives. ".. admonition:: notes" is used in a few places where we had
> an ordered list of multiple notes that would not make sense as
> standalone/separate admonitions.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> Acked-by: Stefan Hajnoczi <stefanha@redhat.com> [for block*.json]
[...]
> diff --git a/qapi/block-core.json b/qapi/block-core.json
> index df5e07debd2..cacedfb771c 100644
> --- a/qapi/block-core.json
> +++ b/qapi/block-core.json
[...]
> @@ -6048,9 +6048,9 @@
> #
> # @name: the name of the internal snapshot to be created
> #
> -# Notes: In transaction, if @name is empty, or any snapshot matching
> -# @name exists, the operation will fail. Only some image formats
> -# support it, for example, qcow2, and rbd.
> +# .. note:: In transaction, if @name is empty, or any snapshot matching
> +# @name exists, the operation will fail. Only some image formats
> +# support it, for example, qcow2, and rbd.
> #
> # Since: 1.7
> ##
[...]
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, (continued)
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, John Snow, 2024/06/20
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, John Snow, 2024/06/20
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, Markus Armbruster, 2024/06/21
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, John Snow, 2024/06/21
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, Markus Armbruster, 2024/06/22
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, John Snow, 2024/06/20
- Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, Markus Armbruster, 2024/06/21
Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, Markus Armbruster, 2024/06/19
Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST,
Markus Armbruster <=
Re: [PATCH 09/13] qapi: convert "Note" sections to plain rST, Markus Armbruster, 2024/06/21
[PATCH 11/13] qapi: add markup to note blocks, John Snow, 2024/06/18
[PATCH 12/13] qapi/parser: don't parse rST markup as section headers, John Snow, 2024/06/18
[PATCH 13/13] qapi: convert "Example" sections to rST, John Snow, 2024/06/18