Re: [VOTE] September 2022 PR of the Month

2022-09-30 Thread Michael Robinson
With four votes, this month's PR of the Month is: [26400] Create a more efficient airflow dag set command https://github.com/apache/airflow/pull/26400 Thanks to all who voted and commented! > On Sep 27, 2022, at 11:29, Michael Robinson > wrote: >

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Felix Uellendall
m/apache/airflow/pull/23592 >>> >>> From: Bas Harenslak >>> Reply-To: "dev@airflow.apache.org" >>> Date: Tuesday, September 27, 2022 at 2:13 PM >>> To: "dev@airflow.apache.org" >>> Subject: RE: [EXTERNAL][VOTE] September 2022

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Kaxil Naik
rit : > >> Same here. Less code, more happiness 😊 >> https://github.com/apache/airflow/pull/23592 >> >> >> >> >> >> *From: *Bas Harenslak >> *Reply-To: *"dev@airflow.apache.org" >> *Date: *Tuesday, September 27, 2022 at 2:13 PM

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Jeambrun Pierre
s Harenslak > *Reply-To: *"dev@airflow.apache.org" > *Date: *Tuesday, September 27, 2022 at 2:13 PM > *To: *"dev@airflow.apache.org" > *Subject: *RE: [EXTERNAL][VOTE] September 2022 PR of the Month > > > > *CAUTION*: This email originated from outside of t

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Beck, Vincent
Same here. Less code, more happiness 😊 https://github.com/apache/airflow/pull/23592 From: Bas Harenslak Reply-To: "dev@airflow.apache.org" Date: Tuesday, September 27, 2022 at 2:13 PM To: "dev@airflow.apache.org" Subject: RE: [EXTERNAL][VOTE] September 2022 PR of the M

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread John Thomas
@ja...@potiuk.com I feel like the Heuristic is more of a conversation starter here. A lot of what makes a PR important is hard to quantify, and so I think trying to make it perfect is a losing proposition. J

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Constance Martineau
ore 'as dag' is really nice. > > > -- > *From:* Bas Harenslak > *Sent:* Tuesday, September 27, 2022 11:11 AM > *To:* dev@airflow.apache.org > *Subject:* RE: [EXTERNAL][VOTE] September 2022 PR of the Month > > > *CAUTION*: This email

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Ferruzzi, Dennis
I'd second this one. https://github.com/apache/airflow/pull/23592 no more 'as dag' is really nice. From: Bas Harenslak Sent: Tuesday, September 27, 2022 11:11 AM To: dev@airflow.apache.org Subject: RE: [EXTERNAL][VOTE] September 2022 PR of the

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Jarek Potiuk
My vote goes for https://github.com/apache/airflow/pull/26290 - Apply PEP-563 (Postponed Evaluation of Annotations) to core airflow Indeed. Lots of good choices. And somehow I have a feeling that our heuristics need an improvement to propose those that we know are important "by heart" :) J On Tu

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Jed Cunningham
Tough month, a lot of good choices!

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Bas Harenslak
My vote goes to https://github.com/apache/airflow/pull/23592 (no more “as dag” needed) > On 27 Sep 2022, at 19:52, Daniel Standish > wrote: > > One vote for https://github.com/apache/airflow/pull/26400 >

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Daniel Standish
One vote for https://github.com/apache/airflow/pull/26400 (improved test command) On Tue, Sep 27, 2022 at 10:50 AM Jed Cunningham wrote: > My write-in is ExternalPythonOperator: > https://github.com/apache/airflow/pull/25780 >

Re: [VOTE] September 2022 PR of the Month

2022-09-27 Thread Jed Cunningham
My write-in is ExternalPythonOperator: https://github.com/apache/airflow/pull/25780

[VOTE] September 2022 PR of the Month

2022-09-27 Thread Michael Robinson
Hey devlist! It’s time again to select a PR of the Month for the Airflow newsletter. The candidate PRs below have been selected using the `get_important_pr_candidates.py` script in airflow/dev/stats, which we’re continuing to tweak. Most recently, we added a #protm tag counter to the score cal