Update Analysis Services Autorest config (ReadMe.md) to include version 2017-08-01-beta#1542
Update Analysis Services Autorest config (ReadMe.md) to include version 2017-08-01-beta#1542michael-falk wants to merge 1 commit intoAzure:currentfrom michael-falk:patch-1
Conversation
|
@michael-falk, |
|
The CI build fails for a file I did not change... |
|
@michael-falk, |
|
Hi There, I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result: File: AutoRest Linter Guidelines | AutoRest Linter Issues Send feedback and make AutoRest Linter Azure Bot smarter day by day! Thanks for your co-operation. |
|
covered #1526 |
|
Hi There, I am the AutoRest Linter Azure bot. I am here to help. My task is to analyze the situation from the AutoRest linter perspective. Please review the below analysis result: File: AutoRest Linter Guidelines | AutoRest Linter Issues Send feedback and make AutoRest Linter Azure Bot smarter day by day! Thanks for your co-operation. |
veronicagg
left a comment
There was a problem hiding this comment.
Looks good, just one question, which once confirmed, I can approve and merge.
Sorry it took me a while, I had to fix CI (per the unrelated error you were seeing).
Thanks!
| ``` yaml | ||
| openapi-type: arm | ||
| tag: package-2016-05 | ||
| tag: package-2017-08 |
There was a problem hiding this comment.
@michael-falk is the beta spec expected to be your new default package? Just asking since I saw this update.
|
Closing per conversation with @taiwu , who indicated this change is included in the linked PR. thanks! |
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information
api-versionin the path should match theapi-versionin the spec).Quality of Swagger