potiuk edited a comment on issue #12587:
URL: https://github.com/apache/airflow/issues/12587#issuecomment-737997145


   > <img alt="Screen Shot 2020-12-02 at 7 17 36 PM" width="437" 
src="https://user-images.githubusercontent.com/2644098/100959466-2a753400-34d3-11eb-958a-feeb5bae4b9f.png";>
   > 
   > crazy that it shows up in my IDE but not in our static checks.
   
   It is likely, because we have  "dagrun.py" still excluded from pylint in 
pylint_todo.txt
   
   ```
   ./airflow/models/dag.py
   ./airflow/models/dagrun.py
   ./airflow/www/utils.py
   ./airflow/configuration.py
   ./airflow/models/taskinstance.py
   ```
   
   Until we remove all the  unneeded 'cyclic dependencies' (note! cyclic 
dependencies, not cyclic imports!) which I am boasting about for quite some 
time, we cannot remove those from the list.
   
   The root cause of it is the way how inter-twined those "core" classes are 
and how they depend on each other in all directions. I hope I will give it a 
shot after 2.0 again.
   
   


----------------------------------------------------------------
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:
[email protected]


Reply via email to