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

add directive for cognitiveservices #31940

Merged
merged 2 commits into from
Dec 20, 2024
Merged

add directive for cognitiveservices #31940

merged 2 commits into from
Dec 20, 2024

Conversation

kazrael2119
Copy link
Contributor

@kazrael2119 kazrael2119 commented Dec 20, 2024 •
edited
Loading

temporary fix as #30491 (comment)

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

openapi-pipeline-app bot commented Dec 20, 2024 •
edited
Loading

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.

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

openapi-pipeline-app bot commented Dec 20, 2024 •
edited
Loading

Generated ApiView

Language Package Name ApiView Link
Go sdk/resourcemanager/cognitiveservices/armcognitiveservices There is no API change compared with the previous version
JavaScript @azure/arm-cognitiveservices https://apiview.dev/Assemblies/Review/a5c5802052af4553bdae74a523c883bd?revisionId=59512c6dc15840bf9a2ef322319f7731
Java azure-resourcemanager-cognitiveservices There is no API change compared with the previous version
.Net Azure.ResourceManager.CognitiveServices There is no API change compared with the previous version
Python azure-mgmt-cognitiveservices https://apiview.dev/Assemblies/Review/0e28ee9c469a48049a651419d2c0c67a?revisionId=45ba4846a70d45e68c9c6e496c0a9774

@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 20, 2024 •
edited
Loading

API change check

APIView has identified API level changes in this PR and created following API reviews.

AnomalyDetector
ContentSafety
Language.Conversations.Authoring
Language.AnalyzeDocuments
Language.Text
Language.Text.Authoring
Language.Conversations
Azure.OpenAI
Microsoft.Speech.VideoTranslation

@AzureRestAPISpecReview AzureRestAPISpecReview added ARMReview ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Dec 20, 2024
This was referenced Dec 20, 2024
Closed
Closed
Closed
Closed
Closed
Closed
@msyyc msyyc self-requested a review December 20, 2024 03:53
@MaryGao MaryGao added PublishToCustomers Acknowledgement the changes will be published to Azure customers. and removed WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required ARMReview labels Dec 20, 2024
@MaryGao MaryGao merged commit 1cb540f into main Dec 20, 2024
29 checks passed
@MaryGao MaryGao deleted the add-dire-for-cog branch December 20, 2024 05:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MaryGao MaryGao MaryGao approved these changes

@qiaozha qiaozha Awaiting requested review from qiaozha qiaozha is a code owner

@msyyc msyyc Awaiting requested review from msyyc

Assignees
No one assigned
Labels
Cognitive Services PublishToCustomers Acknowledgement the changes will be published to Azure customers. ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@kazrael2119 @azure-sdk @MaryGao @msyyc @AzureRestAPISpecReview