Re: cfbot update: Using GitHub for patch review

From: Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>
To: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Cc: Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: cfbot update: Using GitHub for patch review
Date: 2024-06-29 08:42:23
Message-ID: CAGECzQQvsKSSy-YxqSFggj8BBMVRq99fOD=8UdpWhbBP-2FLhQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, 29 Jun 2024 at 01:13, Thomas Munro <thomas(dot)munro(at)gmail(dot)com> wrote:
>
> On Sat, Jun 29, 2024 at 1:10 AM Ashutosh Bapat
> <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> > I need to sign in to github to add my review comments. So those who do not have a github account can not use it for review. But I don't think that can be fixed. We need a way to know who left review comments.
>
> I don't think Jelte was talking about moving review discussion to
> Github, just providing a link to *view* the patches there.

Totally correct. And I realize now I should have called that out
explicitly in the initial email.

While I personally would love to be able to read & write comments on a
Github PR, integrating that with the mailing list in a way that the
community is happy with as a whole is no small task (both technically
and politically).

So (for now) I took the easy way out and sidestepped all those
difficulties, by making the github branches of the cfbot (which we
already had) a bit more user friendly as a way to access patches in a
read-only way.

> Now I'm
> wondering if there is a way to disable comments on commits in the
> postgresql-cfbot GH account. I guess they'd be lost after 48 hours
> anyway when the branch gets force-pushed and commit hash changes? I
> don't want people to start posting comments there that no one is
> looking at.

It seems you can disable them for 6 months at a time here:
https://github.com/postgresql-cfbot/postgresql/settings/interaction_limits

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jelte Fennema-Nio 2024-06-29 09:24:32 Re: Converting README documentation to Markdown
Previous Message Alexander Lakhin 2024-06-29 07:00:00 Re: LogwrtResult contended spinlock