You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2023/03/04 20:55:58 UTC

[GitHub] [airflow] potiuk commented on a diff in pull request #29850: Fixing broken filter in in /taskinstance/list view

potiuk commented on code in PR #29850:
URL: https://github.com/apache/airflow/pull/29850#discussion_r1125553823


##########
airflow/models/taskinstance.py:
##########
@@ -139,6 +139,12 @@
     from airflow.models.operator import Operator
     from airflow.utils.task_group import MappedTaskGroup, TaskGroup
 
+    # This is a workaround because mypy doesn't work with hybrid_property
+    # TODO: remove this hack and move hybrid_property back to main import block

Review Comment:
   ```suggestion
       # TODO: remove this hack and move hybrid_property back to main import block
       # See https://github.com/python/mypy/issues/4430
   ```



-- 
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-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org