sunank200 commented on PR #53389: URL: https://github.com/apache/airflow/pull/53389#issuecomment-3168805732
> > TriggerRule should be part of airflow-shared as it is used in core, SDK, and providers. > > This is just an enum right? In which case it should be added to the extra schema section in the Execution API FastAPI app, that will let it be generated without code sharing, and then providers should try to use the version from the SDK, falling back to the version in core for older Airflow versions. @ashb I have added to the extra schema section in the Execution API FastAPI app > Is this PR complete? There are tons of occurences of trigger_rule which needs to be updated. > > I see it used across example dags, sdk, providers etc. > > Providers need to use the imports from task sdk So do example dags @amoghrajesh I have done the change suggested -- 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]
