Taragolis commented on code in PR #35475:
URL: https://github.com/apache/airflow/pull/35475#discussion_r1383046497


##########
airflow/serialization/serialized_objects.py:
##########
@@ -80,9 +80,9 @@
 
     HAS_KUBERNETES: bool
     try:
-        from kubernetes.client import models as k8s
+        from kubernetes.client import models as k8s  # noqa: TCH004
 
-        from airflow.providers.cncf.kubernetes.pod_generator import 
PodGenerator
+        from airflow.providers.cncf.kubernetes.pod_generator import 
PodGenerator  # noqa: TCH004

Review Comment:
   The real dark magic happen here
   
   
https://github.com/apache/airflow/blob/64a64abd793c5e7eb49ef68e8724c9346c7536d4/airflow/serialization/serialized_objects.py#L446-L448
   
   That is only affect `k8s` and `PodGenerator` so better exclude only this 
lines from check rather then entire module



-- 
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]

Reply via email to