-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
[Compute] CDRP SDK https://github.com/Azure/azure-rest-api-specs/pull/18351 #27726
Conversation
@m-nash can you help with the CI failure? Also this is generated from preview Swagger, should we merge it to main? |
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.
Need to change the API tag in https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/compute/Microsoft.Azure.Management.Compute/AzSdk.RP.props
sdk/compute/Microsoft.Azure.Management.Compute/src/Properties/AssemblyInfo.cs
Outdated
Show resolved
Hide resolved
@archerzz This is supposed to be preview SDK, if I change API tag it will show in consequent stable SDK's. Is that fine? |
@haagha It's OK to have preview api tag. You can see them in other SDKs as well, like this. |
/check-enforcer evaluate |
/check-enforcer reset |
* Initial commit. * update version * adding preview * assembly version change * fix compute_resource-manager * fix resource_manager * assembly changes * tags and assembly version * version update
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.