I am trying to take a stab at fixing the script for PROTM if there's any issue there. The script keeps running forever for me (or I am running out of patience too soon), trying to check it out.
I will reach out to Briana if that's the case and try to figure it out. Thanks & Regards, Amogh Desai On Tue, Oct 29, 2024 at 6:02 PM Amogh Desai <amoghdesai....@gmail.com> wrote: > 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 >> > >> >