[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:35:32 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Markus Armbruster <armbru@redhat.com> writes:
> John Snow <jsnow@redhat.com> writes:
[...]
>> diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json
>> index b3de1fb6b3a..57598331c5c 100644
>> --- a/qga/qapi-schema.json
>> +++ b/qga/qapi-schema.json
[...]
>> @@ -631,8 +632,8 @@
>> # Errors:
>> # - If hybrid suspend is not supported, Unsupported
>> #
>> -# Notes: It's strongly recommended to issue the guest-sync command
>> -# before sending commands when the guest resumes
>> +# .. note:: It's strongly recommended to issue the guest-sync command
>> +# before sending commands when the guest resumes.
>> #
>> # Since: 1.1
>> ##
>> @@ -1461,16 +1462,15 @@
>> # * POSIX: as defined by os-release(5)
>> # * Windows: contains string "server" or "client"
>> #
>> -# Notes: On POSIX systems the fields @id, @name, @pretty-name,
>> -# @version, @version-id, @variant and @variant-id follow the
>> -# definition specified in os-release(5). Refer to the manual page
>> -# for exact description of the fields. Their values are taken
>> -# from the os-release file. If the file is not present in the
>> -# system, or the values are not present in the file, the fields
>> -# are not included.
>> +# .. note:: On POSIX systems the fields @id, @name, @pretty-name,
>> +# @version, @version-id, @variant and @variant-id follow the
>> +# definition specified in os-release(5). Refer to the manual page for
>> +# exact description of the fields. Their values are taken from the
>> +# os-release file. If the file is not present in the system, or the
>> +# values are not present in the file, the fields are not included.
>> #
>> -# On Windows the values are filled from information gathered from
>> -# the system.
>> +# On Windows the values are filled from information gathered from
>> +# the system.
>
> Please don't change the indentation here. I get the same output with
>
> @@ -1461,7 +1462,7 @@
> # * POSIX: as defined by os-release(5)
> # * Windows: contains string "server" or "client"
> #
> -# Notes: On POSIX systems the fields @id, @name, @pretty-name,
> +# .. note:: On POSIX systems the fields @id, @name, @pretty-name,
> # @version, @version-id, @variant and @variant-id follow the
> # definition specified in os-release(5). Refer to the manual page
> # for exact description of the fields. Their values are taken
I'm blind. Actually, you change indentation of subsequent lines from 4
to 3 *everywhere*. I guess you do that to make subsequent lines line up
with the directive, here "note".
Everywhere else, we indent such lines by 4. Hmm. How terrible would it
be not to mess with the alignment?
If we want to use 3 for directives, is it worth pointing out in the
commit message?
[...]
- Re: [PATCH 06/13] docs/qapidoc: fix nested parsing under untagged sections, (continued)
- [PATCH 07/13] qapi: fix non-compliant JSON examples, John Snow, 2024/06/18
- [PATCH 08/13] qapi: ensure all errors sections are uniformly typset, John Snow, 2024/06/18
- [PATCH 10/13] qapi: update prose in note blocks, John Snow, 2024/06/18
- [PATCH 09/13] qapi: convert "Note" sections to plain rST, John Snow, 2024/06/18
- 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, 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