-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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] adding filter property for Autocomplete in SDK #5822
[Azure Search] adding filter property for Autocomplete in SDK #5822
Conversation
ishansrivastava90
commented
Apr 16, 2019
- includes generated code from swagger spec from azure-rest-api-specs:master
- added custom code to AutocompleteParameters for POST flavor of the API
- added tests for the property
@brjohnstmsft FYI. |
src/SDKs/Search/DataPlane/Search.Tests/Tests/AutocompleteTests.cs
Outdated
Show resolved
Hide resolved
src/SDKs/Search/DataPlane/Search.Tests/Tests/AutocompleteTests.cs
Outdated
Show resolved
Hide resolved
src/SDKs/Search/DataPlane/Search.Tests/Tests/AutocompleteTests.cs
Outdated
Show resolved
Hide resolved
@ishansrivastava90 The build failure appears to be a bug in another Azure service's SDK. I'll merge from master again to resolve it. Sorry for the inconvenience. |
Ran autorest on spec changes from the azure master
3b70f2e
to
d710892
Compare
Will merge once builds are passing |
@brjohnstmsft @dsgouda The CI/Travis-CI check is failing consistently with the message, "E: Failed to fetch https://packages.microsoft.com/ubuntu/14.04/prod/dists/trusty/main/binary-amd64/Packages.gz Hash Sum mismatch Any ideas as to get this to pass? |
@ishansrivastava90 We're taking a look |
@dsgouda Both @brjohnstmsft and I are going to be OOF for the 1st part of the next week. If the issue is determined and fixed before we are back, feel free to merge the PR. |
@ishansrivastava90 Please pull lkatest changes from master and update this PR, that should fix the CI issues |