potiuk commented on PR #41637: URL: https://github.com/apache/airflow/pull/41637#issuecomment-2301797352
> Instead of a pre-commit check and enforcing the style of the message did you consider adding extra (required) args to the deprecated decorator Pre-commit check is still needed regardless - because we want to make sure that all deprecations have it. But yes - they **could** be better structured. one problem with adding new fields - is that since providers and airflow core are separated, anything "common" we come up with will have to go to "common.compat" - especially in the light that providers should be compatible with both Airflow 2 and Airflow 3. -- 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]
