From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: Thomas Huth <thuth@redhat.com>,
qemu-devel@nongnu.org, Peter Maydell <peter.maydell@linaro.org>
Subject: Re: [PULL 5/6] docs: split the CI docs into two files
Date: Mon, 30 Aug 2021 23:15:35 +0200 [thread overview]
Message-ID: <4da56a6a-a3b6-f17b-a9a7-c81fe5b903d2@redhat.com> (raw)
In-Reply-To: <f09c711a-3d74-ca4f-05e4-0d6ebf380e5e@redhat.com>
On 8/30/21 6:02 PM, Thomas Huth wrote:
> On 30/08/2021 15.33, Philippe Mathieu-Daudé wrote:
>> Hi Thomas,
>>
>> On 8/25/21 1:00 PM, Thomas Huth wrote:
>>> From: Daniel P. Berrangé <berrange@redhat.com>
>>>
>>> This splits the CI docs into one file talking about job setup and usage
>>> and another file describing provisioning of custom runners.
>>>
>>> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
>>> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>>> Reviewed-by: Willian Rampazzo <willianr@redhat.com>
>>> Message-Id: <20210812180403.4129067-2-berrange@redhat.com>
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>> docs/devel/ci-jobs.rst | 40 ++++++++++
>>> docs/devel/ci-runners.rst | 117 ++++++++++++++++++++++++++++
>>> docs/devel/ci.rst | 159 +-------------------------------------
>>> 3 files changed, 159 insertions(+), 157 deletions(-)
>>> create mode 100644 docs/devel/ci-jobs.rst
>>> create mode 100644 docs/devel/ci-runners.rst
>>
>> You took this one but not the following "docs: add definitions of
>> terms for CI/testing", is there something missing there?
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg830839.html
>
> I took the patch from Daniel's series here:
>
> https://lists.gnu.org/archive/html/qemu-devel/2021-08/msg02102.html
>
> The second patch there did not apply anymore.
Willian can you respin it please?
> Sorry for missing Willian's patch, I can put it in my next PR if nobody
> beats me to it.
No worry, thanks!
next prev parent reply other threads:[~2021-08-30 21:19 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-08-25 11:00 [PULL 0/6] documentation updates Thomas Huth
2021-08-25 11:00 ` [PULL 1/6] docs/about: Remove the duplicated doc Thomas Huth
2021-08-25 11:00 ` [PULL 2/6] docs/about: Unify the subject format Thomas Huth
2021-08-25 11:00 ` [PULL 3/6] docs/about: Add the missing release record in the subject Thomas Huth
2021-08-25 11:00 ` [PULL 4/6] docs/about/removed-features: Move some CLI options to the right location Thomas Huth
2021-08-25 11:00 ` [PULL 5/6] docs: split the CI docs into two files Thomas Huth
2021-08-30 13:33 ` Philippe Mathieu-Daudé
2021-08-30 16:02 ` Thomas Huth
2021-08-30 21:15 ` Philippe Mathieu-Daudé [this message]
2021-08-31 5:07 ` Thomas Huth
2021-08-25 11:00 ` [PULL 6/6] docs: make sphinx-build be quiet by default Thomas Huth
2021-08-25 20:09 ` [PULL 0/6] documentation updates Peter Maydell
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=4da56a6a-a3b6-f17b-a9a7-c81fe5b903d2@redhat.com \
--to=philmd@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.com \
/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).