-
Notifications
You must be signed in to change notification settings - Fork 5.6k
[Resources] Update Microsoft.Resources swagger to match service behavior #8400
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
This reverts commit dc74330.
|
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-go - Release
|
azure-sdk-for-java - Release
|
azure-sdk-for-js - Release
|
azure-sdk-for-net - Release
|
azure-sdk-for-python - Release
|
|
Azure Pipelines successfully started running 1 pipeline(s). |
2 similar comments
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
@Tiano2017, can you please review the change if it matches with current service behavior? |
majastrz
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.
Signed off from ARM side.
|
@NullMDR , there's compile error for SDK Java, can you please have a look? |
|
@ChenTanyi Could you please help on the Java SDK? |
|
@NullMDR It seems not the generation error. Compile error would be fixed manually in SDK release. |
Thanks @ChenTanyi @NullMDR , I will merge this. |
…ior (Azure#8400) * Update swagger for missing properties * Revert "Update swagger for missing properties" This reverts commit dc74330. * Update swagger for missing properties * Update model schema * Change model name * Mark response body properties as readOnly;remove template property
…ior (Azure#8400) * Update swagger for missing properties * Revert "Update swagger for missing properties" This reverts commit dc74330. * Update swagger for missing properties * Update model schema * Change model name * Mark response body properties as readOnly;remove template property
Latest improvements:
This PR includes changes that will match the swagger with what service returns in the response body.
Changes are:
Contribution checklist:
ARM API Review Checklist
Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.
Please follow the link to find more details on API review process.