potiuk commented on a change in pull request #11102:
URL: https://github.com/apache/airflow/pull/11102#discussion_r493540410



##########
File path: .pre-commit-config.yaml
##########
@@ -202,6 +202,12 @@ repos:
           - --convention=pep257
           - --add-ignore=D100,D102,D104,D105,D107,D200,D205,D400,D401
         exclude: 
^tests/.*\.py$|^scripts/.*\.py$|^dev|^backport_packages|^kubernetes_tests
+  - repo: https://gitlab.com/pycqa/flake8

Review comment:
       Is not it the case that flake requires to have the full Airflow 
installed as a local virtualenv locally to be able to run all the Type checks? 
I think you need to have all the dependencies installed in the same versions 
that are in the CI  and all the dependencies have to be installed in the right 
versions.. 




----------------------------------------------------------------
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