Lee-W commented on code in PR #74695: URL: https://github.com/apache/airflow/pull/74695#discussion_r4273157144
########## providers/common/ai/docs/changelog.rst: ########## @@ -44,6 +44,14 @@ Changelog OpenSandbox server configuration, or pass ``SandboxSpec(block_network=False)`` for a sandbox that may reach the network. See :ref:`the OpenSandbox backend <sandbox-backend-opensandbox>`. +.. note:: + ``HookToolset`` no longer accepts the deprecated ``tool_name_prefix`` parameter. Pass + ``tool_prefix`` instead: ``tool_name_prefix="s3_"`` becomes ``tool_prefix="s3"``, since + ``HookToolset`` adds the underscore itself. ``MCPToolset``'s ``tool_prefix`` now defaults to + ``""``, like every other toolset's and like ``MCPHook``'s. Review Comment: Reworded as suggested. Thanks! -- 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]
