zach-overflow commented on issue #59871: URL: https://github.com/apache/airflow/issues/59871#issuecomment-3697396395
Regarding point number 1 to remove ORM references in `0015_2_9_0_update_trigger_kwargs_type.py`, I've added some comparison notes [here](https://github.com/apache/airflow/pull/59239#issuecomment-3697382607). The TL;DR of the link is that it appears we can safely refactor the script to not rely on the `Trigger` class reference, while still resulting in identical SQL statements for the upgrade and downgrade ops. If that linked change is accepted as is, then I think the only remaining work will be the pre-commit hooks, which I'm happy to add in a separate PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
