bugraoz93 commented on code in PR #69535:
URL: https://github.com/apache/airflow/pull/69535#discussion_r3545542450
##########
airflow-core/.pre-commit-config.yaml:
##########
@@ -134,11 +134,16 @@ repos:
require_serial: true
- id: lint-json-schema
name: Lint config_templates/config.yml
- entry: ../scripts/ci/prek/lint_json_schema.py
+ entry: python ../scripts/ci/prek/lint_json_schema.py
Review Comment:
Why we need Python here as Python defined as language. Do we have exception
for this hook not working without it similar to others?
##########
chart/.pre-commit-config.yaml:
##########
@@ -97,12 +97,17 @@ repos:
require_serial: true
- id: lint-json-schema
name: Lint chart/values.schema.json
- entry: ../scripts/ci/prek/lint_json_schema.py
+ entry: python ../scripts/ci/prek/lint_json_schema.py
Review Comment:
Same here and I think for all hooks :)
##########
log3.txt:
##########
@@ -0,0 +1,493 @@
+2026-07-08T15:24:26.106059Z DEBUG prek: 0.4.8 (cccc61bef 2026-07-04)
Review Comment:
This as well please. Not a good practice to push the logs maybe snippets in
PR description would be good if these wanted to be shared
##########
log2.txt:
##########
@@ -0,0 +1,445 @@
+2026-07-08T15:21:03.720432Z DEBUG prek: 0.4.8 (cccc61bef 2026-07-04)
Review Comment:
This should be deleted
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
