all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Léo Le Bouter" <lle-bout@zaclys.net>
To: guix-devel@gnu.org
Cc: Christopher Baines <mail@cbaines.net>
Subject: Semi-automated patch review
Date: Mon, 05 Apr 2021 23:52:53 +0200	[thread overview]
Message-ID: <7ca1c6c292d4defb23c69d019fe9af9877b521dc.camel@zaclys.net> (raw)

[-- Attachment #1: Type: text/plain, Size: 1306 bytes --]

Hello!

Cbaines already runs automated patch testing infra at 
https://data.guix-patches.cbaines.net/ and 
https://patches.guix-patches.cbaines.net/project/guix-patches/list/

Considering that posting robot messages with test/lint/+ result
information on the issues directly and on the ML might get spammy, I
suggest that Cbaines could setup something that sends off-list to all
the participants or just the poster of the patch being tested, as well
as another list like guix-ci-bot@cbaines.net that reviewers could
voluntarily subscribe to to receive all those off-list messages.

Another suggestion is that the infrastructure by Cbaines could include
an easy way to lookup CI information from a bug id and that a link to
see such CI information could be linked to from Mumi's
(issues.gnu.guix.org) UI. But I also really think that mailing the
contributors privately is very important so they can get automated
feedback and save us time without any additional setup or knowledge
required.

What do you think?

I think it's really important that we move forward on this topic where
honestly lots of things have been achieved especially by Cbaines but
with poor visibility so no actual usage to aid the review process when
we would crucially need such help.

Thank you!

Léo

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

             reply	other threads:[~2021-04-05 21:53 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-05 21:52 Léo Le Bouter [this message]
2021-04-06  7:43 ` Semi-automated patch review Christopher Baines
2021-04-07 15:00 ` Andreas Enge
2021-04-07 15:48   ` Vincent Legoll
2021-04-09 11:27   ` Léo Le Bouter
2021-04-09 18:07     ` Vincent Legoll

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=7ca1c6c292d4defb23c69d019fe9af9877b521dc.camel@zaclys.net \
    --to=lle-bout@zaclys.net \
    --cc=guix-devel@gnu.org \
    --cc=mail@cbaines.net \
    /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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.