-
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
Check Resource Name added to swagger #9185
Conversation
You don't have permission to trigger SDK Automation. |
Can one of the admins verify this patch? |
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Outdated
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Outdated
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Outdated
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Outdated
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Outdated
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Show resolved
Hide resolved
...fication/resources/resource-manager/Microsoft.Resources/stable/2020-01-01/subscriptions.json
Outdated
Show resolved
Hide resolved
/azp run automation - sdk |
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-go - Release
|
Trenton Generation - Release
No readme.md specification configuration files were found that are associated with the files modified in this pull request, or swagger_to_sdk section in readme.md is not configured
|
azure-sdk-for-python - Release
|
azure-sdk-for-net - Release
|
/azp run automation - sdk |
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-python-track2 - Release
|
@@ -616,6 +616,12 @@ input-file: | |||
- Microsoft.Authorization/stable/2016-12-01/policyAssignments.json | |||
- Microsoft.Resources/stable/2016-06-01/subscriptions.json | |||
- Microsoft.Resources/stable/2018-05-01/resources.json | |||
|
|||
override-info: | |||
title: PolicyClient |
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.
It should have one value for title
. Can you refer to other tag sections?
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.
The incorrect value in readme.md caused the SDK task failure.
@jianyexi , can you help with the lintdiff failures? |
1 override the title in readme, since there are multiple input swaggers
|
@shreyanshn , did you figure out how to fix the lint failures? |
@raych1 I cannot fix the incompatible values for the operation part of the swagger because it is not a part of the API that I changed. Not sure how to fix that. |
@raych1 I have ran lint locally and no new lint error reported , so I think it can be force merged
|
/azp run automation - sdk |
Azure Pipelines successfully started running 1 pipeline(s). |
Thanks @jianyexi ! |
@akning-ms , as synced with you offline, lintDiff fails by autorest error on different description value for same property which is legacy file. Jianye ran the lintDiff offline and the result looks good for the change. Please help force merge this PR. |
@nullMDR @changlong-liu for awareness track 2 python sdk generation fail for this PR |
* Check Resource Name added to swagger * spellcheck and prettier check fixed * fixes * prettier check and x-ms fixes * added schema for resource name * fixes * description added to resourcename * added changes to all api versions * default response added * prettier checl * default response changed * removed 2016-06-01 api * override errors * file moved to examples * single override * reverting * override policyclient Co-authored-by: Shreyansh Nawlakha <shnawlak@microsoft.com>
Microsoft.Resources/checkResourceName/action is not defined in swagger #8780
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.