Wow, I wasn't aware of the PR: https://github.com/apache/airflow/pull/42943.
Removing those many lines is extremely brave. My vote goes for that one too! Thanks & Regards, Amogh Desai On Tue, Oct 29, 2024 at 12:55 AM Jarek Potiuk <ja...@potiuk.com> wrote: > I was sure there were a few more candidates this month - and I am surprised > to not see them on the list (despite marking them with #protm - we > should likely look at the script :) ). > > My personal candidate (and my vote goes to): > https://github.com/apache/airflow/pull/42943 "Remove BackfillJobRunner > class" => with -3500 lines of code. > > > > On Mon, Oct 28, 2024 at 8:12 PM Briana Okyere > <briana.oky...@astronomer.io.invalid> wrote: > > > Hey All, > > > > It’s once again time to vote for the PR of the Month! > > > > With the help of the `get_important_pr_candidates` script in dev/stats, > > we've identified the following candidates: > > > > PR #42782: AIP-84 Migrate get connections to FastAPI API < > > https://github.com/apache/airflow/pull/42782> > > > > PR #43040: Masking configuration values irrelevant to DAG author < > > https://github.com/apache/airflow/pull/43040> > > > > PR #42080: Fix consistent return response from PubSubPullSensor < > > https://github.com/apache/airflow/pull/42080> > > > > PR #42959: AIP-84 | Public list tags API < > > https://github.com/apache/airflow/pull/42959> > > > > PR #42629: AIP-84 Migrate views /object/historical_metrics_data to Fast > API > > <https://github.com/apache/airflow/pull/42629> > > > > Please reply to this thread with your selection or offer your own > > nominee(s). > > > > Voting will close on Friday, Nov. 1st at 10 AM PST. The winner(s) will be > > featured in the next issue of the Airflow newsletter. > > > > Also, if there’s an article or event that you think should be included in > > this or a future issue of the newsletter, please drop me a line at < > > briana.oky...@astronomer.io> > > > > -- > > Briana Okyere > > Community Manager > > Astronomer > > >
