potiuk commented on issue #58983: URL: https://github.com/apache/airflow/issues/58983#issuecomment-3613353425
Yeah. We fixed it this way and it works ->https://github.com/apache/airflow/pull/59048 The main reason was that we are using some linting rules that expect unused imports that are only used for type checking to be moved to TYPE_CHECKING, but in case of FastAPI this is not really the case, as you are using runtime evaluations of types. I think we will have to look at our ruff rules and decide if (and to what extent) we want to do the TYPE_CHECKING import checks. Thanks for the super-quick fixes @YuriiMotov! You rock there in FastAPI team. -- 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]
