kaxil commented on PR #72939: URL: https://github.com/apache/airflow/pull/72939#issuecomment-6098129852
Toolsets external-services tracking moved off provider.yaml's schema and the shared Sphinx directive entirely. It's a hand-maintained table in supported_services.rst now, and four drift tests in test_provider_metadata.py catch a missing entry, a stale one, or a broken doc target. The dangling file citation in the module docstring is gone too, with the derivation documented at each constant instead. GitHub Copilot and vLLM are reachable through the pydanticai connection and now listed. OpenAI Codex stays off because it only reads OAuth credentials the hook has no way to pass, and TypeSafe landed with the version floor bumped to 2.45 to match. `test_hook_incompatible_auth_modules_stay_incompatible` duplicates the provider-class-discovery logic from `test_deprecated_upstream_modules_stay_deprecated` almost line for line, worth collapsing into one helper at some point. The TypeSafe commit also sneaks in the `ObjectStorageToolset` table entry that a867f664's own drift test should have caught when that commit landed, not here, though it's a one-line fix either way. Good to merge from me. -- 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]
