Received-SPF: pass (google.com: domain of linux-kernel+bounces-98892-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Message-ID: <82cf3fba-fdfa-4185-b2b9-c8ed8d123136@leemhuis.info>
Date: Mon, 11 Mar 2024 14:39:46 +0100
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 1/2] docs: *-regressions.rst: Use collon after regzbot
 introduced command
Content-Language: en-US, de-DE
To: =?UTF-8?B?TsOtY29sYXMgRi4gUi4gQS4gUHJhZG8=?= <nfraprado@collabora.com>,
 Jonathan Corbet <corbet@lwn.net>
Cc: kernel@collabora.com, regressions@lists.linux.dev,
 linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
 workflows@vger.kernel.org
References: <20240308-regzbot-fixes-v1-0-577a4fe16e12@collabora.com>
 <20240308-regzbot-fixes-v1-1-577a4fe16e12@collabora.com>
From: Thorsten Leemhuis <regressions@leemhuis.info>
In-Reply-To: <20240308-regzbot-fixes-v1-1-577a4fe16e12@collabora.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Thx for this!

On 08.03.24 15:09, Nícolas F. R. A. Prado wrote:
> All the examples in the reference documentation for regzbot have a
> collon

s/collon/colon/ here and a few lines below as well. And in the subject
as well. Speaking of which: something like "docs: *-regressions.rst:
add colon to regzbot commands" might be better.

> after the "introduced" command, while on the kernel documentation
> some have and others don't. This suggests both are acceptable,

Yup.

> but in
> order to avoid confusion, add collons after all the commands to match
> the reference docs.

Yeah, good idea. I likely would have done this myself soon while doing a
few other changes I plan, but whatever. :-D

> Link: https://gitlab.com/knurd42/regzbot/-/blob/main/docs/reference.md
> Signed-off-by: Nícolas F. R. A. Prado <nfraprado@collabora.com>

With the changes above:

Reviewed-by: Thorsten Leemhuis <linux@leemhuis.info>

Side note: I wonder if the commit message could come a bit quicker to
the point (something along the lines of "Use colons as command
terminator everywhere for consistency, even if it not strictly
necessary. That way it will also match regzbot's reference
documentation.". But not really important I guess. Up to John.

Ciao, Thorsten

> ---
>  Documentation/admin-guide/reporting-regressions.rst |  2 +-
>  Documentation/process/handling-regressions.rst      | 10 +++++-----
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/admin-guide/reporting-regressions.rst b/Documentation/admin-guide/reporting-regressions.rst
> index d8adccdae23f..76b246ecf21b 100644
> --- a/Documentation/admin-guide/reporting-regressions.rst
> +++ b/Documentation/admin-guide/reporting-regressions.rst
> @@ -31,7 +31,7 @@ The important bits (aka "TL;DR")
>     Linux kernel regression tracking bot "regzbot" track the issue by specifying
>     when the regression started like this::
>  
> -       #regzbot introduced v5.13..v5.14-rc1
> +       #regzbot introduced: v5.13..v5.14-rc1
>  
>  
>  All the details on Linux kernel regressions relevant for users
> diff --git a/Documentation/process/handling-regressions.rst b/Documentation/process/handling-regressions.rst
> index 5d3c3de3f4ec..42b13f77b019 100644
> --- a/Documentation/process/handling-regressions.rst
> +++ b/Documentation/process/handling-regressions.rst
> @@ -27,11 +27,11 @@ The important bits (aka "The TL;DR")
>     is optional, but recommended):
>  
>      * For mailed reports, check if the reporter included a line like ``#regzbot
> -      introduced v5.13..v5.14-rc1``. If not, send a reply (with the regressions
> +      introduced: v5.13..v5.14-rc1``. If not, send a reply (with the regressions
>        list in CC) containing a paragraph like the following, which tells regzbot
>        when the issue started to happen::
>  
> -       #regzbot ^introduced 1f2e3d4c5b6a
> +       #regzbot ^introduced: 1f2e3d4c5b6a
>  
>      * When forwarding reports from a bug tracker to the regressions list (see
>        above), include a paragraph like the following::
> @@ -79,7 +79,7 @@ When doing either, consider making the Linux kernel regression tracking bot
>  "regzbot" immediately start tracking the issue:
>  
>   * For mailed reports, check if the reporter included a "regzbot command" like
> -   ``#regzbot introduced 1f2e3d4c5b6a``. If not, send a reply (with the
> +   ``#regzbot introduced: 1f2e3d4c5b6a``. If not, send a reply (with the
>     regressions list in CC) with a paragraph like the following:::
>  
>         #regzbot ^introduced: v5.13..v5.14-rc1
> @@ -398,9 +398,9 @@ By using a 'regzbot command' in a direct or indirect reply to the mail with the
>  regression report. These commands need to be in their own paragraph (IOW: they
>  need to be separated from the rest of the mail using blank lines).
>  
> -One such command is ``#regzbot introduced <version or commit>``, which makes
> +One such command is ``#regzbot introduced: <version or commit>``, which makes
>  regzbot consider your mail as a regressions report added to the tracking, as
> -already described above; ``#regzbot ^introduced <version or commit>`` is another
> +already described above; ``#regzbot ^introduced: <version or commit>`` is another
>  such command, which makes regzbot consider the parent mail as a report for a
>  regression which it starts to track.
>  
>