potiuk commented on code in PR #46891:
URL: https://github.com/apache/airflow/pull/46891#discussion_r2211831396
##########
.pre-commit-config.yaml:
##########
@@ -1014,7 +1014,12 @@ repos:
name: Update Airflow's meta-package pyproject.toml
language: python
entry: ./scripts/ci/pre_commit/update_airflow_pyproject_toml.py
- files:
^.*/pyproject\.toml$|^scripts/ci/pre_commit/update_airflow_pyproject_toml\.py$
+ files: >
+ (?x)
+ ^.*/pyproject\.toml$|
+ ^scripts/ci/pre_commit/update_airflow_pyproject_toml\.py$|
+ ^providers/.*/pyproject\.toml$|
+ ^providers/.*/provider\.yaml$
Review Comment:
Yes. I found it as part of this PR, that we should trigger the update when
provider definiton change. Changing only provider.yaml files in the way that it
would affect how airflow metadata project's pyproject.toml is defined (say
Pyton 3.13 exclusion added) just failed static checks in the PR, where it runs
them with `--all-files`
--
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]
