gopidesupavan commented on code in PR #63763:
URL: https://github.com/apache/airflow/pull/63763#discussion_r2943588875


##########
.pre-commit-config.yaml:
##########
@@ -1063,10 +1046,20 @@ repos:
           ^airflow-core/src/airflow/api_fastapi/core_api/datamodels/.*\.py$|
           
^airflow-core/src/airflow/api_fastapi/auth/managers/simple/(datamodels|routes|services|openapi)/.*\.py$
         require_serial: true
-        ## ONLY ADD PREK HOOKS HERE THAT REQUIRE CI IMAGE
       - id: check-contextmanager-class-decorators
         name: Check for problematic context manager class decorators
         entry: ./scripts/ci/prek/check_contextmanager_class_decorators.py
         language: python
         files: .*test.*\.py$
         pass_filenames: true
+      - id: update-uv-lock

Review Comment:
   probably adding a comment in the pre-commit file on top of this hook would 
be better if someone moves this or adds any hooks after this might lead extra 
issues ?



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

Reply via email to