insomnes commented on code in PR #46603:
URL: https://github.com/apache/airflow/pull/46603#discussion_r1948216067
##########
.pre-commit-config.yaml:
##########
@@ -764,7 +761,7 @@ repos:
name: Verify usage of Airflow deprecation classes in core
entry: category=DeprecationWarning|category=PendingDeprecationWarning
files: \.py$
- exclude:
^airflow/configuration\.py$|^providers/src/airflow/providers/|^scripts/in_container/verify_providers\.py$|^(providers/)?tests/.*$|^tests_common/
+ exclude:
^airflow/configuration\.py$|^providers/.*/src/airflow/providers/|^scripts/in_container/verify_providers\.py$|^(providers/)?tests/.*$|^tests_common/
Review Comment:
I am not an expert, but I think, the deprecation exclusion expansion from
`^(providers/)?tests/.*$` which previously covered tests in providers might
need some adjustment for the new structure: `providers/<name>/tests/...`.
Something like `^(providers/)?tests/.*$|^(providers/.*/tests/.*$` may work.
--
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]
