-
Notifications
You must be signed in to change notification settings - Fork 5.6k
adjust csharp SDK generation for Microsoft.AVS #9773
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
Conversation
[Staging] Swagger Validation Report️~[Staging] BreakingChange [Detail]Posted by Swagger Pipeline |
|
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-js - Release
|
azure-sdk-for-go - Release
|
Azure CLI Extension Generation - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-net - Release
|
azure-sdk-for-python - Release
|
Trenton Generation - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-java - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
|
Can one of the admins verify this patch? |
|
@isra-fel, could you take a look at this PR? |
isra-fel
left a comment
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.
LGTM
|
@isra-fel, thanks for approving. Do you have the permissions to merge it? I do not. |
|
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-python-track2 - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
|
@lirenhe or @isra-fel , can you review the most recent changes and merge the pull request? The generated code may be viewed in Azure/azure-sdk-for-net#12602 |
|
I don't have the permission to merge, but the changes look good to me |
|
There is a conflict of readme.csharp.md, @ctaggart, could you help to resolve it? |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@lirenhe, please merge this. |
* adjust csharp generation for Microsoft.AVS * renamed namespace to "Avs" & client is "AvsClient"
No description provided.