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