linux-doc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: teng sterling <sterlingteng@gmail.com>
To: Jonathan Corbet <corbet@lwn.net>
Cc: Yanteng Si <siyanteng@loongson.cn>, Alex Shi <alexs@kernel.org>,
	Huacai Chen <chenhuacai@kernel.org>,
	Jiaxun Yang <jiaxun.yang@flygoat.com>,
	linux-doc@vger.kernel.org, realpuyuwang@gmail.com,
	"Wu X.C." <bobwxc@email.cn>, Yanteng Si <siyanteng01@gmail.com>
Subject: Re: [PATCH v7 00/11] docs/zh_CN: add core api and openrisc translation
Date: Wed, 21 Apr 2021 09:34:44 +0800	[thread overview]
Message-ID: <CAMU9jJohxSfv4kKD8rCDqeXentA9Yqan0NVNR2JfYYD=aKmXWg@mail.gmail.com> (raw)
In-Reply-To: <87lf9cip3m.fsf@meer.lwn.net>

OK!
Maybe I misunderstood the patch split。~>_<~
And next time:
when I add a new file, I will do the associated index.rst changes in
the same patch.

Thanks,

Yanteng

Jonathan Corbet <corbet@lwn.net> 于2021年4月21日周三 上午6:11写道:
>
> Yanteng Si <siyanteng@loongson.cn> writes:
>
> > This series of patches translates core-api/irq/* and openrisc/* into Chinese.
> >
> > v1 -> v2:
> >
> > Modify some words under Xiangcheng Wu's suggestion. Thanks for his rewiew!
> >
> > v2 -> v3:
> >
> > keep the signs number equal to the title char under Alex's suggestion.
> > Thanks for his rewiew!
> >
> > v3 -> v4:
> >
> > Add Xiangcheng Wu's review sign to 0002、0003 and 0008; Thanks for his rewiew!
> >
> > v4 -> v5:
> >
> > Split 1-7 into a separate series.
> >
> > v5 -> v6:
> >
> > Modify the Subject of patches 07 and 11.
> > Patch 11 will not be sent if not modified. (name repeat)
> >
> > Revert to one series because there is no need to split into two series.
> >
> > v6 -> v7:
> >
> > Send patch 11 to the series. Thank you Xiangcheng for your help!
> >
> > Yanteng Si (11):
> >   docs/zh_CN: add core-api irq concepts.rst translation
> >   docs/zh_CN: add core-api irq irq-affinity.rst translation
> >   docs/zh_CN: add core-api irq irq-domain.rst translation
> >   docs/zh_CN: add core-api irq irqflags-tracing.rst translation
> >   docs/zh_CN: add core-api irq index.rst translation
> >   docs/zh_CN: add core-api index.rst translation
> >   docs/zh_CN: add core api translation to zh_CN index
> >   docs/zh_CN: add openrisc openrisc_port.rst translation
> >   docs/zh_CN: add openrisc todo.rst translation
> >   docs/zh_CN: add openrisc index.rst translation
> >   docs/zh_CN: add openrisc translation to zh_CN index
>
> I have applied this series, thanks.
>
> I have a request for the future, though: when you add a new file, do the
> associated index.rst changes *in the same patch*.  Among other things,
> that helps bisectability - we'll know which patch adds a build error,
> for example.  It's not hugely important for docs, but is a good way to
> approach kernel work in general - don't stack up lots of work that only
> becomes active as the result of a small patch at the end.
>
> THanks,
>
> jon

      reply	other threads:[~2021-04-21  1:34 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-16 10:32 [PATCH v7 00/11] docs/zh_CN: add core api and openrisc translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 01/11] docs/zh_CN: add core-api irq concepts.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 02/11] docs/zh_CN: add core-api irq irq-affinity.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 03/11] docs/zh_CN: add core-api irq irq-domain.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 04/11] docs/zh_CN: add core-api irq irqflags-tracing.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 05/11] docs/zh_CN: add core-api irq index.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 06/11] docs/zh_CN: add core-api " Yanteng Si
2021-04-16 10:32 ` [PATCH v7 07/11] docs/zh_CN: add core api translation to zh_CN index Yanteng Si
2021-04-16 10:32 ` [PATCH v7 08/11] docs/zh_CN: add openrisc openrisc_port.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 09/11] docs/zh_CN: add openrisc todo.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 10/11] docs/zh_CN: add openrisc index.rst translation Yanteng Si
2021-04-16 10:32 ` [PATCH v7 11/11] docs/zh_CN: add openrisc translation to zh_CN index Yanteng Si
2021-04-16 11:58   ` Wu X.C.
2021-04-20 22:10 ` [PATCH v7 00/11] docs/zh_CN: add core api and openrisc translation Jonathan Corbet
2021-04-21  1:34   ` teng sterling [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAMU9jJohxSfv4kKD8rCDqeXentA9Yqan0NVNR2JfYYD=aKmXWg@mail.gmail.com' \
    --to=sterlingteng@gmail.com \
    --cc=alexs@kernel.org \
    --cc=bobwxc@email.cn \
    --cc=chenhuacai@kernel.org \
    --cc=corbet@lwn.net \
    --cc=jiaxun.yang@flygoat.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=realpuyuwang@gmail.com \
    --cc=siyanteng01@gmail.com \
    --cc=siyanteng@loongson.cn \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).