Hey All, Thank you so much for your feedback on this. It sounds like we are good to move forward with our current script that features the Top 5 PRs of the Month, and then voting for our top 3 from those.
On Fri, Oct 27, 2023 at 12:51 PM Aritra Basu <aritrabasu1...@gmail.com> wrote: > I'm not sure on the exact heuristics used by the script but how about all > PRs with atleast 1 (or 2 or a specific number) votes from the ones picked > by the script gets featured. > > -- > Regards, > Aritra Basu > > On Sat, Oct 28, 2023, 1:04 AM Jarek Potiuk <ja...@potiuk.com> wrote: > > > Yeah. Top PRs of the month sound good. > > > > On Fri, Oct 27, 2023 at 6:16 PM Amogh Desai <amoghdesai....@gmail.com> > > wrote: > > > > > 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/> > > > > > > > > > > > > > > >
