-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
Bring Back action UpdateHostname and UploadCertificate #1861
Conversation
Hi There, I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result: 💡 Please review potentially introduced Error(s)/Warning(s): Analysis Report 💡 File: AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback Thanks for your co-operation. |
@sarangan12 ping! |
@solankisamir Please take a look at https://travis-ci.org/Azure/azure-rest-api-specs/jobs/287657498. Your changes have caused 2 new errors which must be fixed. Please fix them and update PR |
@sarangan12 I will fix the description issue, but the other issue around certificate_password naming, we had to bring back, as we need that resource in our .NET Client, which gets integrated to powershell. To make a breaking change in powershell, we need it around for one more version. |
Opened issue on self #1881 |
Hi There, I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result: 💡 Please review potentially introduced Error(s)/Warning(s): Analysis Report 💡 File: AutoRest Linter Guidelines | AutoRest Linter Issues | Send feedback Thanks for your co-operation. |
@sarangan12 ping! on this. Let me know, if I am not clear. |
No modification for AutorestCI/azure-sdk-for-python |
No modification for AutorestCI/azure-sdk-for-node |
No modification for AutorestCI/azure-sdk-for-ruby |
We need to support updatehostname and uploadcertificate actions for one more version, to avoid customers breaking change in powershell which is built on .NET client.
These actions will be deprecated in next apiversion
PR information
api-version
in the path should match theapi-version
in the spec).Quality of Swagger