From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
---|---|
To: | Sehrope Sarkuni <sehrope(at)jackdb(dot)com> |
Cc: | pgsql-www(at)lists(dot)postgresql(dot)org |
Subject: | Re: Add maling list message counts to Commitfest app |
Date: | 2019-07-23 12:36:50 |
Message-ID: | CE15A5E6-E82F-4A19-A0F1-11B1677BB19A@yesql.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-www |
> On 23 Jul 2019, at 14:22, Sehrope Sarkuni <sehrope(at)jackdb(dot)com> wrote:
>
> On Tue, Jul 23, 2019 at 3:20 AM Daniel Gustafsson <daniel(at)yesql(dot)se <mailto:daniel(at)yesql(dot)se>> wrote:
> What insights do you envision that information would provide to users working
> with the app?
>
> Came about while I've been going through the Commitfest app looking for something to pickup as a first time reviewer. I figure mailing list message counts are a decent proxy for how much / how many people have weighed in on a particular patch and how complicated it may be.
I don’t think there is a reliable correlation between the length of the thread
and the complexity of the patch (the existance of bikeshedding as a concept
kind of proves that).
> Stats from the latest patches would be useful as well (total size, number of files changed, lines added, lines removed, etc). That'd be more complicated though as would need something to parse the patch and deal with multiple attachments.
LoC is also not a very truthful metric for complexity, but I agree that would
no doubt the nice to have (although the list of changed files is the part that
appeals to me). It is as you say slightly more complicated though.
I agree that it would be nice if patches suitable for beginners where marked in
order to lower the barrier to entry, but I don’t have any answers on how to do
that.
cheers ./daniel
From | Date | Subject | |
---|---|---|---|
Next Message | Sehrope Sarkuni | 2019-07-23 18:44:15 | Re: Add maling list message counts to Commitfest app |
Previous Message | Sehrope Sarkuni | 2019-07-23 12:22:45 | Re: Add maling list message counts to Commitfest app |