ashb commented on issue #6596: [AIRFLOW-6004] Untangle Executors class to avoid cyclic imports URL: https://github.com/apache/airflow/pull/6596#issuecomment-560015370 > > Please could we split all the unrelated change out (mostly all the `# pylint:x` to `pylint: x` comments). Is the first wrong, or are we just favouring one over the other? If so we should also add a lint rule to enforce this style. > > Sure. Some of the changes were just really helpful with untangling the dependencies (but specifically the "pylint" ones was a byproduct). But now it should be easy to separate out some of those changes into separate commits. I will do it shortly. Thank you very much - I ask because I've used `git bisect` a couple of times over the last few weeks on Airflow to track down bugs already :)
---------------------------------------------------------------- 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
