ashb commented on pull request #12466: URL: https://github.com/apache/airflow/pull/12466#issuecomment-735376106
> Both providers_manager and plugins_manager are at the same "logical" level, so there is no reason why providers manager should import anything from plugins_manager. Why not plugins manager importing the same function from providers manager ? > > So extracting it to common module makes perfect sense and is "cleaner" architecture - and this is what pylint complains about. Yes, in this case I agree it is cleaner. My general complaint with this pylint check is that in general I disagree that it detects actual problems. ---------------------------------------------------------------- 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]
