-
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
Add new API version 2019-07-01 for SIG swagger spec #6400
Conversation
SDK Automation [Logs] (Generated from 46c41c8, Iteration 8)
|
Automation for azure-sdk-for-rubyNothing to generate for azure-sdk-for-ruby |
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
Automation for azure-sdk-for-goA PR has been created for you based on this PR content. Once this PR will be merged, content will be added to your service PR: |
Automation for azure-sdk-for-javaNothing to generate for azure-sdk-for-java |
Can one of the admins verify this patch? |
@AutorestCI regenerate azure-sdk-for-go |
Two properties changed to writable from readonly. Signing off from ARM side. |
...ource-manager/Microsoft.Compute/stable/2019-07-01/examples/CreateOrUpdateASimpleGallery.json
Outdated
Show resolved
Hide resolved
...e/resource-manager/Microsoft.Compute/stable/2019-07-01/examples/GetAGalleryImageVersion.json
Show resolved
Hide resolved
specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/gallery.json
Outdated
Show resolved
Hide resolved
LGTM |
@jerryc3 this looks good from SDK perspective. Does anybody else need to review/sign off? |
Hi Joel, Thanks for the review! The code owners Pengfei/Hyonho still need to take a look. I also got informed yesterday that my team is trying to add another small field. I think it would be better to add it to this PR so SDK team only has to regenerate once - I will update this PR as soon as it is ready. |
specification/compute/resource-manager/Microsoft.Compute/stable/2019-07-01/gallery.json
Show resolved
Hide resolved
LGTM |
@jerryc3 is this ready now? |
This is ready! I have asked Hyonho to look at it and then we can merge as soon as he signs off. |
LGTM. Thanks. |
@jhendrixMSFT Got all required approvals. Thanks! |
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
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.