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
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 |