You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/05/22 11:37:25 UTC

[GitHub] [airflow] ashb edited a comment on pull request #8962: [AIRFLOW-8057] [AIP-31] Add @task decorator

ashb edited a comment on pull request #8962:
URL: https://github.com/apache/airflow/pull/8962#issuecomment-632646688


   > The main question is: do we want to support that? Is this "hack" worth it, and do IDEs get massively confused by this?
   
   I think if IDEs support it, and we add tests for it I'd say, yes.
   
   I wonder if we can do this in `airflow/__init__.py` (adding the task decorator import)
   
   ```python
   STATICA_HACK = True
   globals()['kcah_acitats'[::-1].upper()] = False
   if STATICA_HACK:  # pragma: no cover
       from airflow.models.dag import DAG
       from airflow.exceptions import AirflowException
       from airflow.decorators import task
   ```
   
   Without confusing IDEs more.


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

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