All of lore.kernel.org
 help / color / mirror / Atom feed
* Enabling Sashiko replies to linux-crypto?
@ 2026-10-20 21:57 Eric Biggers
  2026-10-20 22:12 ` Jason A. Donenfeld
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Eric Biggers @ 2026-10-20 21:57 UTC (permalink / raw)
  To: linux-crypto, Herbert Xu, Ard Biesheuvel, Jason A. Donenfeld

Currently, Sashiko is configured (in
https://github.com/sashiko-dev/sashiko/blob/main/projects/linux/mailing_lists.toml)
to monitor linux-crypto for patches, but not to send emails to it.

Would there be any objections to me opening a pull request that changes
it to the same setting that I set the fscrypt and fsverity lists to?

    ["linux-crypto@vger.kernel.org"]
    reply_to = ["author", "list", "recipients"]

That would enable emails for issues found, with reply-all.

(There's also a 'positive_review = "all"' setting that would enable
emails for positive reviews, i.e. "Sashiko has reviewed this patch and
found no issues. It looks great!"  But those are kind of noisy.)

- Eric

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Enabling Sashiko replies to linux-crypto?
  2026-10-20 21:57 Enabling Sashiko replies to linux-crypto? Eric Biggers
@ 2026-10-20 22:12 ` Jason A. Donenfeld
  2026-10-21  2:53 ` Herbert Xu
  2026-10-21  6:18 ` Thomas Huth
  2 siblings, 0 replies; 6+ messages in thread
From: Jason A. Donenfeld @ 2026-10-20 22:12 UTC (permalink / raw)
  To: Eric Biggers; +Cc: linux-crypto, Herbert Xu, Ard Biesheuvel

I was looking to do this on the wireguard mailing list too. (I have
some review comments on a PR I need to address for that.) No
objections from me.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Enabling Sashiko replies to linux-crypto?
  2026-10-20 21:57 Enabling Sashiko replies to linux-crypto? Eric Biggers
  2026-10-20 22:12 ` Jason A. Donenfeld
@ 2026-10-21  2:53 ` Herbert Xu
  2026-10-21  6:18 ` Thomas Huth
  2 siblings, 0 replies; 6+ messages in thread
From: Herbert Xu @ 2026-10-21  2:53 UTC (permalink / raw)
  To: Eric Biggers; +Cc: linux-crypto, Ard Biesheuvel, Jason A. Donenfeld

On Tue, Oct 20, 2026 at 09:57:44PM +0000, Eric Biggers wrote:
> Currently, Sashiko is configured (in
> https://github.com/sashiko-dev/sashiko/blob/main/projects/linux/mailing_lists.toml)
> to monitor linux-crypto for patches, but not to send emails to it.
> 
> Would there be any objections to me opening a pull request that changes
> it to the same setting that I set the fscrypt and fsverity lists to?
> 
>     ["linux-crypto@vger.kernel.org"]
>     reply_to = ["author", "list", "recipients"]
> 
> That would enable emails for issues found, with reply-all.
> 
> (There's also a 'positive_review = "all"' setting that would enable
> emails for positive reviews, i.e. "Sashiko has reviewed this patch and
> found no issues. It looks great!"  But those are kind of noisy.)

Sure that sounds like a good idea.

Thanks,
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Enabling Sashiko replies to linux-crypto?
  2026-10-20 21:57 Enabling Sashiko replies to linux-crypto? Eric Biggers
  2026-10-20 22:12 ` Jason A. Donenfeld
  2026-10-21  2:53 ` Herbert Xu
@ 2026-10-21  6:18 ` Thomas Huth
  2026-10-21  6:54   ` Eric Biggers
  2 siblings, 1 reply; 6+ messages in thread
From: Thomas Huth @ 2026-10-21  6:18 UTC (permalink / raw)
  To: Eric Biggers, linux-crypto, Herbert Xu, Ard Biesheuvel,
	Jason A. Donenfeld

On 20/10/2026 23.57, Eric Biggers wrote:
> Currently, Sashiko is configured (in
> https://github.com/sashiko-dev/sashiko/blob/main/projects/linux/mailing_lists.toml)
> to monitor linux-crypto for patches, but not to send emails to it.
> 
> Would there be any objections to me opening a pull request that changes
> it to the same setting that I set the fscrypt and fsverity lists to?
> 
>      ["linux-crypto@vger.kernel.org"]
>      reply_to = ["author", "list", "recipients"]
> 
> That would enable emails for issues found, with reply-all.
> 
> (There's also a 'positive_review = "all"' setting that would enable
> emails for positive reviews, i.e. "Sashiko has reviewed this patch and
> found no issues. It looks great!"  But those are kind of noisy.)
Is it possible to configure it in a way that it only sends mails if it 
really found something? ... always getting "Sashiko has reviewed this patch 
and it looks great" mails is rather annoying than helpful ...

  Thomas


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Enabling Sashiko replies to linux-crypto?
  2026-10-21  6:18 ` Thomas Huth
@ 2026-10-21  6:54   ` Eric Biggers
  2026-10-21  7:25     ` Thomas Huth
  0 siblings, 1 reply; 6+ messages in thread
From: Eric Biggers @ 2026-10-21  6:54 UTC (permalink / raw)
  To: Thomas Huth; +Cc: linux-crypto, Herbert Xu, Ard Biesheuvel, Jason A. Donenfeld

On Wed, Oct 21, 2026 at 08:18:18AM +0200, Thomas Huth wrote:
> On 20/10/2026 23.57, Eric Biggers wrote:
> > Currently, Sashiko is configured (in
> > https://github.com/sashiko-dev/sashiko/blob/main/projects/linux/mailing_lists.toml)
> > to monitor linux-crypto for patches, but not to send emails to it.
> > 
> > Would there be any objections to me opening a pull request that changes
> > it to the same setting that I set the fscrypt and fsverity lists to?
> > 
> >      ["linux-crypto@vger.kernel.org"]
> >      reply_to = ["author", "list", "recipients"]
> > 
> > That would enable emails for issues found, with reply-all.
> > 
> > (There's also a 'positive_review = "all"' setting that would enable
> > emails for positive reviews, i.e. "Sashiko has reviewed this patch and
> > found no issues. It looks great!"  But those are kind of noisy.)
> Is it possible to configure it in a way that it only sends mails if it
> really found something? ... always getting "Sashiko has reviewed this patch
> and it looks great" mails is rather annoying than helpful ...

Yes, that's what I proposed above (and it's merged now).

- Eric

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: Enabling Sashiko replies to linux-crypto?
  2026-10-21  6:54   ` Eric Biggers
@ 2026-10-21  7:25     ` Thomas Huth
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Huth @ 2026-10-21  7:25 UTC (permalink / raw)
  To: Eric Biggers; +Cc: linux-crypto, Herbert Xu, Ard Biesheuvel, Jason A. Donenfeld

On 21/10/2026 08.54, Eric Biggers wrote:
> On Wed, Oct 21, 2026 at 08:18:18AM +0200, Thomas Huth wrote:
>> On 20/10/2026 23.57, Eric Biggers wrote:
>>> Currently, Sashiko is configured (in
>>> https://github.com/sashiko-dev/sashiko/blob/main/projects/linux/mailing_lists.toml)
>>> to monitor linux-crypto for patches, but not to send emails to it.
>>>
>>> Would there be any objections to me opening a pull request that changes
>>> it to the same setting that I set the fscrypt and fsverity lists to?
>>>
>>>       ["linux-crypto@vger.kernel.org"]
>>>       reply_to = ["author", "list", "recipients"]
>>>
>>> That would enable emails for issues found, with reply-all.
>>>
>>> (There's also a 'positive_review = "all"' setting that would enable
>>> emails for positive reviews, i.e. "Sashiko has reviewed this patch and
>>> found no issues. It looks great!"  But those are kind of noisy.)
>> Is it possible to configure it in a way that it only sends mails if it
>> really found something? ... always getting "Sashiko has reviewed this patch
>> and it looks great" mails is rather annoying than helpful ...
> 
> Yes, that's what I proposed above (and it's merged now).
Oh, -ENOTENOUGHCOFFEEYET ... Sorry, I should read more closely before 
answering ;-)

  Thomas


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2026-10-21  7:25 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-20 21:57 Enabling Sashiko replies to linux-crypto? Eric Biggers
2026-10-20 22:12 ` Jason A. Donenfeld
2026-10-21  2:53 ` Herbert Xu
2026-10-21  6:18 ` Thomas Huth
2026-10-21  6:54   ` Eric Biggers
2026-10-21  7:25     ` Thomas Huth

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.