kaxil commented on code in PR #72306: URL: https://github.com/apache/airflow/pull/72306#discussion_r3963780670
########## providers/anthropic/src/airflow/providers/anthropic/hooks/anthropic.py: ########## @@ -71,7 +71,7 @@ DEFAULT_MODEL = "claude-opus-4-8" #: Platforms that serve the first-party-only endpoints (Message Batches, token -#: counting, the Models API). Amazon Bedrock, Google Vertex AI and Microsoft +#: counting, the Models API and Managed Agents). Amazon Bedrock, Google Vertex AI and Microsoft Review Comment: Could this also cover the `aws` member while you're here? The block calls these first-party-only endpoints and names Amazon Bedrock as not serving them, but the set includes `aws`, so the distinction a reader actually needs, Bedrock vs Claude Platform on AWS, is the one the comment never makes; `connections.rst` spells it out as "the first-party Anthropic API (``platform="anthropic"``) and Claude Platform on AWS (``platform="aws"``)", and `_require_first_party`'s error message names both. Rewrapping for that would also pull the new line back from 95 chars to the block's ~78. -- 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]
