BasPH commented on issue #5238: [AIRFLOW-4364] Add Pylint to CI URL: https://github.com/apache/airflow/pull/5238#issuecomment-489460775 @potiuk The script was a bit tricky to get right. Will comment on all individual comments after this. About the path to complete Pylint compatibility: I'd love to have Airflow completely Pylint compatible asap. I think checking only changed lines is the fastest way to have some form of Pylint in Airflow, and then we (the community) can work on making all code compatible in the meantime. Since checking only changed lines is a bit messy and requires these extra scripts, another option is to first make everything compatible, touch 99% of the codebase in a single PR (or split up PRs per module/package) and then enable Pylint in the CI afterwards. What do you think?
---------------------------------------------------------------- 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
