Lee-W commented on code in PR #70499:
URL: https://github.com/apache/airflow/pull/70499#discussion_r3688291664
##########
scripts/in_container/run_provider_yaml_files_check.py:
##########
@@ -489,6 +493,25 @@ def
check_hook_class_name_entries_in_connection_types(yaml_files: dict[str, dict
return num_connection_types, num_errors
+@run_check("Checking that python-modules in retry-policies exist and belong to
provider package")
+def check_retry_policy_modules_exist_and_belong_to_package(yaml_files:
dict[str, dict]) -> tuple[int, int]:
Review Comment:
Reworded to state the rule instead of enumerating categories:
`parse_module_data`'s glob assumes a category's modules live in a directory
named after its yaml key, and retry policies intentionally live under
`policies/` instead — so that check can't be reused and this existence-only
check stands in for it.
--
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]
