Skip to content
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.

Sign up for GitHub

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

Jump to bottom

clean up search specs #31992

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

clean up search specs #31992

wants to merge 1 commit into from

Conversation

xiangyan99
Copy link
Member

@xiangyan99 xiangyan99 commented Jan 3, 2025 •
edited
Loading

Per our API versioning guidelines,

  • deleted Microsoft.Azure.Search.Data folder
  • deleted Microsoft.Azure.Search.Service folder
  • deleted all preview versions except the latest one 2024-11-01-preview
  • fixed typo in readme.md

@openapi-pipeline-app openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Jan 3, 2025 •
edited
Loading

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR has at least one breaking change (label: BreakingChangeReviewRequired).
    To unblock this PR, follow the process at aka.ms/brch.
  • ❌ The required check named Swagger BreakingChange has failed. To unblock this PR, follow the process at aka.ms/brch.

@openapi-pipeline-app openapi-pipeline-app
Copy link

PR validation pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@AzureRestAPISpecReview AzureRestAPISpecReview added BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required data-plane labels Jan 3, 2025
@azure-sdk azure-sdk mentioned this pull request Jan 3, 2025
Draft
@xiangyan99 xiangyan99 added the PublishToCustomers Acknowledgement the changes will be published to Azure customers. label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@arv100kri arv100kri Awaiting requested review from arv100kri arv100kri will be requested when the pull request is marked ready for review arv100kri is a code owner

@bleroy bleroy Awaiting requested review from bleroy bleroy will be requested when the pull request is marked ready for review bleroy is a code owner

@BevLoh BevLoh Awaiting requested review from BevLoh BevLoh will be requested when the pull request is marked ready for review BevLoh is a code owner

@giulianob giulianob Awaiting requested review from giulianob giulianob will be requested when the pull request is marked ready for review giulianob is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required data-plane PublishToCustomers Acknowledgement the changes will be published to Azure customers. Search
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xiangyan99 @AzureRestAPISpecReview