potiuk commented on issue #6929: [AIRFLOW-6373] Make airflow/utils pylint compatible URL: https://github.com/apache/airflow/pull/6929#issuecomment-569419734 That's the point. Those ARE real (not probable) cyclic imports. And of course Pylint reports more/different kinds cyclic imports than python parser - otherwise we would not even get there at the first place because pylint would fail parsing the code. Pylint deliberately reports the cycles that developers have hidden by using local imports. But they are only hidden - they are logical cycles. They are still there but in order to stop python parser from failing at import time developer deferred resolving imports to later phase when method is executed. But real coupling between the MODULE containing the method and imported class remains. It's the logical cycle that is there: resetdb() (from db) method uses 'models' which are using provide_session() (again from db). This is a real cycle. And it does not need to be ther
---------------------------------------------------------------- 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] With regards, Apache Git Services
