Message-ID: <38cb9f23-a56a-f420-5942-0bfeb620306e@gmail.com>
Date:   Tue, 7 Mar 2023 00:13:50 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.7.1
To:     carlos.bilbao@amd.com
Cc:     corbet@lwn.net, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, sergio.collado@gmail.com,
        Akira Yokosawa <akiyks@gmail.com>
References: <20230306134420.30210-1-carlos.bilbao@amd.com>
Subject: Re: [PATCH] docs/sp_SP: Add process deprecated translation
Content-Language: en-US
From:   Akira Yokosawa <akiyks@gmail.com>
In-Reply-To: <20230306134420.30210-1-carlos.bilbao@amd.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Carlos,

Minor nits in the Subject and Sob area.

On Mon, 6 Mar 2023 07:44:20 -0600, Carlos Bilbao wrote:
> Subject: [PATCH] docs/sp_SP: Add process deprecated translation

This summary looks ambiguous to me.

Maybe

   docs/sp_SP: Add translation of process/deprecated 

??

> Translate Documentation/process/deprecated.rst into Spanish.
> 
> Co-developed-by: Carlos Bilbao <carlos.bilbao@amd.com>
> Signed-off-by: Sergio Gonzalez <sergio.collado@gmail.com>
> Signed-off-by: Carlos Bilbao <carlos.bilbao@amd.com>

To me, Co-developed-by: from the author of the patch looks
strange, because it is obvious the author did some development on
the patch.

Which is your intent:

   Author: Carlos
   Co-developer: Sergio

, or

   Author: Sergio
   Co-developer: Carlos

???

        Thanks, Akira

> ---
>  .../translations/sp_SP/process/deprecated.rst | 381 ++++++++++++++++++
>  .../translations/sp_SP/process/index.rst      |   1 +
>  2 files changed, 382 insertions(+)
>  create mode 100644 Documentation/translations/sp_SP/process/deprecated.rst
[...]