lkml.org 
[lkml]   [2022]   [Mar]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [RFC PATCH 2/2] Documentation: dev-tools: use literal block instead of code-block
On Thu, Mar 17, 2022 at 4:26 AM <frowand.list@gmail.com> wrote:
>
> From: Frank Rowand <frank.rowand@sony.com>
>
> KTAP Specification: Change code-block directives to straightforward
> literal blocks since the blocks do not contain code.
>
> Suggested-by: Jonathan Corbet <corbet@lwn.net>
> Signed-off-by: Frank Rowand <frank.rowand@sony.com>
> ---

This looks good to me.

I'd personally rather push this through independently of the KTAP 2.0
spec updates, as it's really just a minor formatting change to the
spec, and it has no impact on the actual KTAP format.

So, if we can accept this independently, that'd be swell.

Reviewed-by: David Gow <davidgow@google.com>

Cheers,
-- David


> Documentation/dev-tools/ktap.rst | 18 ++++++++----------
> 1 file changed, 8 insertions(+), 10 deletions(-)
>
> diff --git a/Documentation/dev-tools/ktap.rst b/Documentation/dev-tools/ktap.rst
> index 37b5dc61bfb8..b9a57ceddd4f 100644
> --- a/Documentation/dev-tools/ktap.rst
> +++ b/Documentation/dev-tools/ktap.rst
> @@ -115,34 +115,32 @@ The diagnostic data field is optional, and results which have neither a
> directive nor any diagnostic data do not need to include the "#" field
> separator.
>
> -Example result lines include:
> -
> -.. code-block:: none
> +Example result lines include::
>
> ok 1 test_case_name
>
> The test "test_case_name" passed.
>
> -.. code-block:: none
> +::
>
> not ok 1 test_case_name
>
> The test "test_case_name" failed.
>
> -.. code-block:: none
> +::
>
> ok 1 test # SKIP necessary dependency unavailable
>
> The test "test" was SKIPPED with the diagnostic message "necessary dependency
> unavailable".
>
> -.. code-block:: none
> +::
>
> not ok 1 test # TIMEOUT 30 seconds
>
> The test "test" timed out, with diagnostic data "30 seconds".
>
> -.. code-block:: none
> +::
>
> ok 5 check return code # rcode=0
>
> @@ -202,7 +200,7 @@ allowed to be either indented or not indented.
>
> An example of a test with two nested subtests:
>
> -.. code-block:: none
> +::
>
> KTAP version 1
> 1..1
> @@ -215,7 +213,7 @@ An example of a test with two nested subtests:
>
> An example format with multiple levels of nested testing:
>
> -.. code-block:: none
> +::
>
> KTAP version 1
> 1..2
> @@ -250,7 +248,7 @@ nested version line, uses a line of the form
>
> Example KTAP output
> --------------------
> -.. code-block:: none
> +::
>
> KTAP version 1
> 1..1
> --
> Frank Rowand <frank.rowand@sony.com>
>
[unhandled content-type:application/pkcs7-signature]
\
 
 \ /
  Last update: 2022-03-17 09:44    [W:0.101 / U:1.432 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site