potiuk commented on code in PR #34247:
URL: https://github.com/apache/airflow/pull/34247#discussion_r1387343224
##########
.pre-commit-config.yaml:
##########
@@ -1030,3 +1030,12 @@ repos:
files:
^airflow/migrations/versions/.*\.py$|^docs/apache-airflow/migrations-ref\.rst$
additional_dependencies: ['rich>=12.4.4']
## ONLY ADD PRE-COMMITS HERE THAT REQUIRE CI IMAGE
+ - id: bandit
Review Comment:
And to add even more: system pre-commit are special - they will **just**
run whatever you put there as a script. This is how we are adding the
pre-commits that are using `breeze` CI image to run stuff - those scripts of
ours will make sure the image is locally built and they will run `docker run`
to run whatever they need to run as part of the check. There the `environment`
managerment responsibility is passed from pre-commit to breeze essentially.
--
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.
To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
