Hi, I also think having multiple PRs under PR of the month would be really nice.
One way to approach this is: What we can do is, collect votes for all the stakeholders for their top 3 PRs in the list as Pierre mentioned and then create score for each PR. The top X PRs from this list goes in the newsletter. Thanks & Regards, Amogh Desai On Fri, Oct 27, 2023, 21:37 Pierre Jeambrun <pierrejb...@gmail.com> wrote: > Hello all, > > I like the idea of highlighting more than just 1 PR in the "PR of the month > section", especially when we have a hard time deciding between a few good > candidates. > > IMHO the script does not always select good candidates or sometimes miss > some good candidates, (because of the heuristic we use) and should still be > reviewed by 'human' and put under a vote. > > An idea would be to keep the current process we have, but vote for lets > say, up to 3 PR per person, and adapt the newsletter to handle a PR batch > of the month. > > (We can also improve the heuristic so we can blindly trust the X top > candidates it outputs, but we are not here yet I believe) > > Le ven. 27 oct. 2023 à 17:56, Briana Okyere > <briana.oky...@astronomer.io.invalid> a écrit : > > > Hey All, > > > > A couple months back, Jed Cunningham proposed that we batch our PRs, > > instead of including just one in the PR of the Month section of the > Airflow > > Newsletter. Thread below: > > > > <https://lists.apache.org/thread/9l663x2910zj5gkohg1mgpxg18ogsyz2> > > > > I'd like to put this up to a vote. Do you prefer we only include 1 PR > each > > month? Or, should we include multiple? Currently, we use a script to pull > > the top 5. Should all 5 be included in each month's issue? > > > > Looking forward to hearing your thoughts prior to the release of the > > October issue next week. > > > > -- > > Briana Okyere > > Community Manager > > Email: briana.oky...@astronomer.io > > <john....@astronomer.io>Mobile: +1 415.713.9943 > > Time zone: US Pacific UTC > > > > <https://www.astronomer.io/> > > >
