+1 for https://github.com/apache/airflow/pull/41916 too.
It is nice how a "first" time contributor has collaborated with the maintainers and managed to solve many issues and merge it. Kudos to https://github.com/Illumaria irrespective of the result! Thanks & Regards, Amogh Desai On Wed, Nov 27, 2024 at 12:18 AM Elad Kalif <elad...@apache.org> wrote: > I vote for Add support for semicolon stripping to DbApiHook, PrestoHook, > and TrinoHook > https://github.com/apache/airflow/pull/41916 > > What started as a small scoped PR turned to be a masive one. Including at > least 3 edge cases discoved with our CI. Noting that the author is first > time contribuor who did not drop it though it took weeks with countless > rebases and review rounds. > > I think this is a great example of valued member of the community that we > should recignize. > > בתאריך יום ג׳, 26 בנוב׳ 2024, 20:27, מאת Briana Okyere > <briana.oky...@astronomer.io.invalid>: > > > 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 #43367: Add Trigger DAG UI with advance options < > > https://github.com/apache/airflow/pull/43367> > > > > PR #41325: Add "@asset" to decorate a function as a DAG and an asset < > > https://github.com/apache/airflow/pull/41325> > > > > PR #43893: Start building the replacement task runner for Task Execution > > SDK <https://github.com/apache/airflow/pull/43893> > > > > PR #42404: Remove logical_date from DAG Run APIs and Functions, > transition > > to run_id as sole identifier for Airflow< > > https://github.com/apache/airflow/pull/42404> > > > > PR #43902: Rename execution_date to logical_date across codebase < > > https://github.com/apache/airflow/pull/43902> > > > > Please reply to this thread with your selection or offer your own > > nominee(s). > > > > Voting will close on Friday, Nov. 29th at 5 PM 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 > > >
