Received-SPF: pass (google.com: domain of linux-kernel+bounces-212267-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
From: Jonathan Corbet <corbet@lwn.net>
To: Dongliang Mu <dzm91@hust.edu.cn>, Alex Shi <alexs@kernel.org>, Yanteng
 Si <siyanteng@loongson.cn>, Haoyang Liu <tttturtleruss@hust.edu.cn>,
 Dongliang Mu <dzm91@hust.edu.cn>, Vegard Nossum <vegard.nossum@oracle.com>
Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] docs/zh_CN: Update dev-tools/index.rst
In-Reply-To: <20240612161835.18931-1-dzm91@hust.edu.cn>
References: <20240612161835.18931-1-dzm91@hust.edu.cn>
Date: Wed, 12 Jun 2024 15:12:57 -0600
Message-ID: <871q51q2zq.fsf@trenco.lwn.net>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain

Dongliang Mu <dzm91@hust.edu.cn> writes:

> Update to commit 8c88bc5b489e ("docs: dev-tools: Add UAPI checker
> documentation")
>
> Signed-off-by: Dongliang Mu <dzm91@hust.edu.cn>
> ---
>  Documentation/translations/zh_CN/dev-tools/index.rst | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/translations/zh_CN/dev-tools/index.rst b/Documentation/translations/zh_CN/dev-tools/index.rst
> index fa900f5beb68..c540e4a7d5db 100644
> --- a/Documentation/translations/zh_CN/dev-tools/index.rst
> +++ b/Documentation/translations/zh_CN/dev-tools/index.rst
> @@ -20,18 +20,22 @@ Documentation/translations/zh_CN/dev-tools/testing-overview.rst
>  
>     testing-overview
>     sparse
> +   kcov
>     gcov
>     kasan
> -   kcov
>     ubsan
>     kmemleak
>     gdb-kernel-debugging
>  
>  Todolist:
>  
> + - checkpatch
>   - coccinelle
> + - kmsan
>   - kcsan
>   - kfence
>   - kgdb
>   - kselftest
>   - kunit/index
> + - ktap
> + - checkuapi

So I have to say that the changelog here is not particularly helpful.
You have *not* updated the translation to that commit, so it doesn't
seem like you should say that you did.  "Add several newish documents to
the todo list" is appropriate for something like this.

I've applied the patch, but with the changed commit message.

Thanks,

jon