[VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Briana Okyere <br...@astronomer.io.INVALID>]

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.okyere@astronomer.io
<jo...@astronomer.io>Mobile: +1 415.713.9943
Time zone: US Pacific UTC

<https://www.astronomer.io/>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Briana Okyere <br...@astronomer.io.INVALID>]

Mixed up my J's! Thank you for catching Jarek :)

On Mon, Oct 30, 2023 at 10:30 AM Jarek Potiuk <ja...@potiuk.com> wrote:

> It was Jed's comment, but yes. I agree with it too :)
>
> On Mon, Oct 30, 2023 at 5:51 PM Briana Okyere
> <br...@astronomer.io.invalid> wrote:
>
> > Thank you Jarek, this is very helpful.
> >
> > With your input in mind, let's move forward with a more flexible
> structure.
> > If there is no clear Top PR, we can feature multiple. If there is, we can
> > just include 1.
> >
> > Also, moving forward, committers and community members alike can bump
> > specific PR's as we review them, making them more likely to be included
> in
> > the script.
> >
> > On Fri, Oct 27, 2023 at 3:37 PM Jed Cunningham <jedcunningham@apache.org
> >
> > wrote:
> >
> > > Just to clarify, I didn't mean we should highlight multiple PRs every
> > > month. In that month none of @eumiro's PRs were individually enough to
> be
> > > highlighted but in bulk they were. That was an unusual situation
> though.
> > >
> > > I think we should be flexible with it. I'm not opposed to having
> multiple
> > > per month, but if we do I think they should be equally notable. If we
> > have
> > > a clear winner, we should just highlight that one PR. My 2c.
> > >
> > > I'll also call out that we (committers, community members, anyone
> > really!)
> > > should toss protm on good candidates as we are reviewing stuff
> > day-to-day.
> > > That bumps up the score in the script and will help ensure the good
> stuff
> > > bubbles to the top of the scripts output! A quick search shows we've
> done
> > > this less than a dozen times so far.
> > >
> >
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Jarek Potiuk <ja...@potiuk.com>]

It was Jed's comment, but yes. I agree with it too :)

On Mon, Oct 30, 2023 at 5:51 PM Briana Okyere
<br...@astronomer.io.invalid> wrote:

> Thank you Jarek, this is very helpful.
>
> With your input in mind, let's move forward with a more flexible structure.
> If there is no clear Top PR, we can feature multiple. If there is, we can
> just include 1.
>
> Also, moving forward, committers and community members alike can bump
> specific PR's as we review them, making them more likely to be included in
> the script.
>
> On Fri, Oct 27, 2023 at 3:37 PM Jed Cunningham <je...@apache.org>
> wrote:
>
> > Just to clarify, I didn't mean we should highlight multiple PRs every
> > month. In that month none of @eumiro's PRs were individually enough to be
> > highlighted but in bulk they were. That was an unusual situation though.
> >
> > I think we should be flexible with it. I'm not opposed to having multiple
> > per month, but if we do I think they should be equally notable. If we
> have
> > a clear winner, we should just highlight that one PR. My 2c.
> >
> > I'll also call out that we (committers, community members, anyone
> really!)
> > should toss protm on good candidates as we are reviewing stuff
> day-to-day.
> > That bumps up the score in the script and will help ensure the good stuff
> > bubbles to the top of the scripts output! A quick search shows we've done
> > this less than a dozen times so far.
> >
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Briana Okyere <br...@astronomer.io.INVALID>]

Thank you Jarek, this is very helpful.

With your input in mind, let's move forward with a more flexible structure.
If there is no clear Top PR, we can feature multiple. If there is, we can
just include 1.

Also, moving forward, committers and community members alike can bump
specific PR's as we review them, making them more likely to be included in
the script.

On Fri, Oct 27, 2023 at 3:37 PM Jed Cunningham <je...@apache.org>
wrote:

> Just to clarify, I didn't mean we should highlight multiple PRs every
> month. In that month none of @eumiro's PRs were individually enough to be
> highlighted but in bulk they were. That was an unusual situation though.
>
> I think we should be flexible with it. I'm not opposed to having multiple
> per month, but if we do I think they should be equally notable. If we have
> a clear winner, we should just highlight that one PR. My 2c.
>
> I'll also call out that we (committers, community members, anyone really!)
> should toss protm on good candidates as we are reviewing stuff day-to-day.
> That bumps up the score in the script and will help ensure the good stuff
> bubbles to the top of the scripts output! A quick search shows we've done
> this less than a dozen times so far.
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Jed Cunningham <je...@apache.org>]

Just to clarify, I didn't mean we should highlight multiple PRs every
month. In that month none of @eumiro's PRs were individually enough to be
highlighted but in bulk they were. That was an unusual situation though.

I think we should be flexible with it. I'm not opposed to having multiple
per month, but if we do I think they should be equally notable. If we have
a clear winner, we should just highlight that one PR. My 2c.

I'll also call out that we (committers, community members, anyone really!)
should toss protm on good candidates as we are reviewing stuff day-to-day.
That bumps up the score in the script and will help ensure the good stuff
bubbles to the top of the scripts output! A quick search shows we've done
this less than a dozen times so far.

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Briana Okyere <br...@astronomer.io.INVALID>]

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 <ar...@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 <am...@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 <pi...@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
> > > > <br...@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.okyere@astronomer.io
> > > > > <jo...@astronomer.io>Mobile: +1 415.713.9943
> > > > > Time zone: US Pacific UTC
> > > > >
> > > > > <https://www.astronomer.io/>
> > > > >
> > > >
> > >
> >
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Aritra Basu <ar...@gmail.com>]

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 <am...@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 <pi...@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
> > > <br...@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.okyere@astronomer.io
> > > > <jo...@astronomer.io>Mobile: +1 415.713.9943
> > > > Time zone: US Pacific UTC
> > > >
> > > > <https://www.astronomer.io/>
> > > >
> > >
> >
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Jarek Potiuk <ja...@potiuk.com>]

Yeah. Top PRs of the month sound good.

On Fri, Oct 27, 2023 at 6:16 PM Amogh Desai <am...@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 <pi...@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
> > <br...@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.okyere@astronomer.io
> > > <jo...@astronomer.io>Mobile: +1 415.713.9943
> > > Time zone: US Pacific UTC
> > >
> > > <https://www.astronomer.io/>
> > >
> >
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Amogh Desai <am...@gmail.com>]

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 <pi...@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
> <br...@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.okyere@astronomer.io
> > <jo...@astronomer.io>Mobile: +1 415.713.9943
> > Time zone: US Pacific UTC
> >
> > <https://www.astronomer.io/>
> >
>

Re: [VOTE] Add Multiple PR's of the Month to the Airflow Newsletter

[Pierre Jeambrun <pi...@gmail.com>]

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
<br...@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.okyere@astronomer.io
> <jo...@astronomer.io>Mobile: +1 415.713.9943
> Time zone: US Pacific UTC
>
> <https://www.astronomer.io/>
>