raphaelauv commented on PR #41047: URL: https://github.com/apache/airflow/pull/41047#issuecomment-2254555507
> Okay .Also all the providers are affected , could you confirm me that the design is okay before I do the big refacto , thanks > > cause I would need to put that lines in every sensors -> > > https://github.com/apache/airflow/blob/f5b2364b4a5d965d07da0f79f834507f716c5de1/airflow/providers/sftp/sensors/sftp.py#L70 > > and that mean not official providers will not have the warning I made a commit 'other design proposition` that have multiple advantages - no need to add provider common.compat in dependancy of all providers - no need to add a proactive call to all existing sensors to _is_at_least_2_10_0 - all non official providers will show a warning from airflow 2.10.0 - no need to convert all sensors to the new parameters right now , a big band is error prone wdyt @eladkal @potiuk thanks -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org