Skip to content

Re-emit from TypeSpec to pick up fix for "type" discriminator in WebSearchApproximateLocation#45276

Open
dargilco wants to merge 3 commits intomainfrom
dargilco/web-search-approximate-location
Open

Re-emit from TypeSpec to pick up fix for "type" discriminator in WebSearchApproximateLocation#45276
dargilco wants to merge 3 commits intomainfrom
dargilco/web-search-approximate-location

Conversation

@dargilco
Copy link
Member

Emit from TypeSpec PR: Azure/azure-rest-api-specs#40594

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR re-emits the azure-ai-projects generated Python models from TypeSpec to incorporate an upstream fix for the "type" discriminator on WebSearchApproximateLocation.

Changes:

  • Updates WebSearchApproximateLocation.type to be a required Literal["approximate"] and ensures it is set during initialization.
  • Removes trailing whitespace-only lines in several web search-related sample docstrings.

Reviewed changes

Copilot reviewed 1 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_web_search_with_custom_search.py Trims whitespace-only docstring line.
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_web_search_preview.py Trims whitespace-only docstring line.
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_bing_grounding.py Trims whitespace-only docstring line.
sdk/ai/azure-ai-projects/samples/agents/tools/sample_agent_bing_custom_search.py Trims whitespace-only docstring line.
sdk/ai/azure-ai-projects/azure/ai/projects/models/_models.py Regenerated model: makes WebSearchApproximateLocation.type a required literal discriminator and assigns "approximate" in __init__.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments