Skip to content
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

test #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

test #34

wants to merge 1 commit into from

Conversation

xiaoxuqi-ms
Copy link
Owner

MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.

Contribution checklist:

If any further question about AME onboarding or validation tools, please view the FAQ.

ARM API Review Checklist

  • Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
    • adding/removing APIs.
    • adding/removing properties.
    • adding/removing API-version.
    • adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

  • If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
    Please follow the link to find more details on API review process.

@xiaoxuqi-ms xiaoxuqi-ms reopened this Jun 17, 2020
@openapi-devapp-for-tiramisuq
Copy link

openapi-devapp-for-tiramisuq bot commented Jun 17, 2020

[Staging] Swagger Validation Report

️✔️BreakingChange [Detail]
 There are no breaking changes :) 
LintDiff: 4 Errors, 0 Warnings [Detail] [Expand]
Rule Message
R3012 - APIVersionPattern API Version must be in the format: yyyy-MM-dd, optionally followed by -preview, -alpha, -beta, -rc, -privatepreview.
New: Microsoft.DBforMySQL/stable/2017-12-01/mysql.json#L6
R4001 - XmsParameterLocation The parameter 'serverName' is defined in global parameters section without 'x-ms-parameter-location' extension. This would add the parameter as the client property. Please ensure that this is exactly you want. If so, apply the extension 'x-ms-parameter-location': 'client'. Else, apply the extension 'x-ms-parameter-location': 'method'.
New: Microsoft.DBforMySQL/stable/2017-12-01/mysql.json#L2693
R4010 - RequiredDefaultResponse The response is defined but without a default error response implementation.Consider adding it.'
New: Microsoft.DBforMySQL/stable/2017-12-01/mysql.json#L315
R4010 - RequiredDefaultResponse The response is defined but without a default error response implementation.Consider adding it.'
New: Microsoft.DBforMySQL/stable/2017-12-01/mysql.json#L1366
🔄Avocado [Detail]
Posted by Swagger Pipeline | How to fix these errors?

@xiaoxuqi-ms
Copy link
Owner Author

[Staging] Swagger Validation Report

️✔️LintDiff [Detail][Expand Details]
️✔️ModelValidatio on [Detail][Expand Details]
️✔️Semantic [Detail][Expand Details]
Download plugin to change Github pages dynamically for better experience.
Posted by Swagger Pipeline

console.log test/bots/pipeline/commentBuilder.test.ts:43

[Staging] Swagger Validation Report

LintDiff: 6 Errors, 11 Warnings [Detail][Expand Details]

Only 10 items are listed, please refer to log for more details.

| Rule | Message |
|------|---------|
|<code><code>❌</code></code> [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|
|<code><code>❌</code></code> [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|
|<code><code>❌</code></code> [R3007 - PutGetPatchResponseSchema](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "For a given path with PUT, GET and PATCH operations, the schema of the response must be the same.")|For a given path with PUT, GET and PATCH operations, the schema of the response must be the same.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|
|<code><code>❌</code></code> [R3013 - DeleteMustNotHaveRequestBody](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "The request body of a delete operation must be empty.")|The request body of a delete operation must be empty.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|
|<code><code>❌</code></code> Error message for raw message|"exitCode":"0",<br>"otherDetails":"test":"error raw message"|
|:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    |:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    |:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    |:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    |:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    </details><details><summary><code>️<code>⚠️</code></code><b>ModelValidation: 3 Warnings</b>  [<a h href="https://github.com/zhenglaizhang/azure-rest-api-specs-only/pull/12/checks?check_run_id=557426884"          target="_blank">Detail</a>][Expand Details]
  </summary><br>

| Rule | Message |
|------|---------|
|<code><code>❌</code></code> Error message for raw message|"exitCode":"-1",<br>"otherDetails":"test":"error raw message"|
|<code><code>❌</code></code> Error message for raw message|"exitCode":"-1",<br>"otherDetails":"test":"error raw message"|
|<code><code>❌</code></code> Error message for raw message|"exitCode":"-1",<br>"otherDetails":"test":"error raw message"|
|:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    |:warning: [R2044 - InvalidVerbUsed](https://github.com/Azure/azure-rest-api-specs/blob/master/documentation/openapi-authoring-automated-guidelines.md#r2044 "Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.")|Each operation definition must have a HTTP verb and it must be DELETE/GET/PUT/PATCH/HEAD/OPTIONS/POST/TRACE.<br>swagger: [Microsoft.DataBox/stable/2019-09-01/databox.json:123:12](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:123:12)<br>example: [Microsoft.DataBox/stable/2019-09-01/databox.json:126:13](specification/databox/resource-manager/Microsoft.DataBox/stable/2019-09-01/databox.json:126:13)|    |:warning: Missing message for raw message|"exitCode":"1",<br>"otherDetails":"test":"warning raw message"|
|:warning: Missing message for raw message|"exitCode":"1",<br>"otherDetails":"test":"warning raw message"|
|:warning: Missing message for raw message|"exitCode":"1",<br>"otherDetails":"test":"warning raw message"|
</details><details><summary><code>️<code>️✔️</code></code><b>Semantic</b>  [<a href="https://githu  ub.com/zhenglaizhang/azure-rest-api-specs-only/pull/12/checks?check_run_id=557426884"
      target="_blank">Detail</a>][Expand Details]
  </summary><pre></pre></details><sub><a href="https://chrome.google.com/webstore/detail/wide-github/kaalofacklcidaampbokdplbklpeldpj"
target="_blank"><b>Download plugin to change Github pages dynamically for better experience.</b></a></sub><br><sub>Posted by <b>Swagger Pipeline</b></sub>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant