eladkal commented on code in PR #55416:
URL: https://github.com/apache/airflow/pull/55416#discussion_r2346709137
##########
.pre-commit-config.yaml:
##########
@@ -231,6 +231,14 @@ repos:
entry: ./scripts/ci/prek/check_deferrable_default.py
pass_filenames: false
files: ^(providers/.*/)?airflow/.*/(sensors|operators)/.*\.py$
+ - id: check-new-airflow-exception-usage
+ name: Check new AirflowException usage
+ language: python
+ entry: ./scripts/ci/prek/check_new_airflow_exception_usage.py
+ pass_filenames: true
+ require_serial: true
+ types: [python]
+ files: ^(providers/|airflow-.*/|task-sdk/).+\.py$
Review Comment:
Not sure I follow how are we handling this?
Do we want to review all existing AirflowException in our code base and only
then activate this pre-commit?
--
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
