moiseenkov commented on PR #29462: URL: https://github.com/apache/airflow/pull/29462#issuecomment-1621318954
> We had a recent change on how we treat default deferrable setting and we need to adjust this one too: #31712 > > One more case in point @Lee-W that we need to add a pre-commit checking the defferrable convention, otherwise we will end up will all kind of mess where some deferrable parameters will have the default values following the convention and some not. > > @VladaZakharova - can you please update the change to follow this patterm? @Lee-W -> will it be possible that you add such pre-commit ([#32355 (comment)](https://github.com/apache/airflow/pull/32355#issuecomment-1621100211)) ? Thanks for the update! I will update PR and ping you afterwards -- 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]
