-
Notifications
You must be signed in to change notification settings - Fork 4.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Azure Search] Fix the definition of WebApiSkill #7080
[Azure Search] Fix the definition of WebApiSkill #7080
Conversation
sdk/search/Microsoft.Azure.Search/tests/Tests/SkillsetsTests.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a concern about an apparently unrelated test change.
sdk/search/Microsoft.Azure.Search/tests/Tests/SkillsetsTests.cs
Outdated
Show resolved
Hide resolved
@Careyjmac and I have added extra context in the thread. Let me know what you think |
LGTM... preceding version bump looks good as well given this is a breaking change. Merging... |
This SDK was generated based on this completed pull request in the swagger repository.
This pull request, intends to correct the definition of the
WebApiSkill
model in the SDK, by making headers a simple dictionary within the skill and not a nested property.Added a test to ensure correctness
This is a breaking change, and is already preceeded by a version bump in this PR
Fixes #6468