-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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 ARM resources specs for 2019-08-01 #6948
Conversation
In Testing, Please Ignore[Logs] (Generated from 70472db, Iteration 5).NET: test-repo-billy/azure-sdk-for-net [Logs] [Diff]
Python: test-repo-billy/azure-sdk-for-python [Logs]
Java: test-repo-billy/azure-sdk-for-java [Logs]
Go: test-repo-billy/azure-sdk-for-go [Logs] [Diff]
JavaScript: test-repo-billy/azure-sdk-for-js [Logs] [Diff]
|
Automation for azure-sdk-for-pythonThe initial PR has been merged into your service PR: |
Can one of the admins verify this patch? |
@anthony-c-martin Hello. Could you please resolve the CI errors? |
Automation for azure-sdk-for-goNothing to generate for azure-sdk-for-go |
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.
Approved from ARMs side
@myronfanqiu ping? |
specification/resources/resource-manager/Microsoft.Resources/stable/2019-08-01/resources.json
Show resolved
Hide resolved
specification/resources/resource-manager/Microsoft.Resources/stable/2019-08-01/resources.json
Show resolved
Hide resolved
specification/resources/resource-manager/Microsoft.Resources/stable/2019-08-01/resources.json
Show resolved
Hide resolved
specification/resources/resource-manager/Microsoft.Resources/stable/2019-08-01/resources.json
Show resolved
Hide resolved
@anthony-c-martin Hello. What's the current status? Should I invite Java SDK owner to check this CI error? Or we are waiting for other PR merged first? |
@myronfanqiu Yes please invite the Java SDK owner to take a look - I'm not able to see what's going on from the logs. |
@yaohaizh Hello. Could you please check the Java CI errors? Looks like an error related with Autorest Java Extention. |
@anthony-c-martin Meanwhile, please solve the CI errors of the lintdiff |
Working on the Java SDK issue. |
specification/resources/resource-manager/Microsoft.Resources/stable/2019-08-01/resources.json
Outdated
Show resolved
Hide resolved
@anthony-c-martin Hello. Please solve the CI errors based on the error log and the suggestions. |
c687c04
to
70472db
Compare
@myronfanqiu mind merging if you're happy with the changes? |
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.