Skip to content
This repository was archived by the owner on Aug 26, 2025. It is now read-only.

[TestUnifiedPipeline] [Hub Generated] Review request for Microsoft.Network to add version stable/2020-11-01#1886

Closed
zhenglaizhang wants to merge 0 commit into
test-repo-billy:masterfrom
disreedh:disreedh-dev-frontdoor-Microsoft.Network-2020-11-01
Closed

[TestUnifiedPipeline] [Hub Generated] Review request for Microsoft.Network to add version stable/2020-11-01#1886
zhenglaizhang wants to merge 0 commit into
test-repo-billy:masterfrom
disreedh:disreedh-dev-frontdoor-Microsoft.Network-2020-11-01

Conversation

@zhenglaizhang
Copy link
Copy Markdown

Mirror from https://github.com/Azure/azure-rest-api-specs/pull/11353

@openapi-pipeline-app-test
Copy link
Copy Markdown

openapi-pipeline-app-test Bot commented Oct 27, 2020

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail]
There are no breaking changes.
️❌~[Staging] LintDiff: 1 Errors, 3 Warnings failed [Detail] [Expand]
Rule Message
R3023 - OperationsAPIImplementation Operations API must be implemented for '/providers/Microsoft.Network/operations'.
New: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L36
⚠️ R4021 - DescriptionAndTitleMissing 'ruleSetAction' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
New: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L625
⚠️ R4021 - DescriptionAndTitleMissing 'ruleSetDefaultAction' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
New: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L765
⚠️ R4021 - DescriptionAndTitleMissing 'ruleSetFeatures' model/property lacks 'description' and 'title' property. Consider adding a 'description'/'title' element. Accurate description/title is essential for maintaining reference documentation.
New: Microsoft.Network/stable/2020-11-01/webapplicationfirewall.json#L769
️⚠️~[Staging] Avocado: 1 Warnings warning [Detail] [Expand]
Rule Message
⚠️ MULTIPLE_API_VERSION The default tag contains multiple API versions swaggers.
readme: specification/frontdoor/resource-manager/readme.md
tag: specification/frontdoor/resource-manager/readme.md#tag-package-2020-11
️️✔️~[Staging] ModelValidation succeeded [Detail]
Validation passes for ModelValidation.
️️✔️~[Staging] SemanticValidation succeeded [Detail]
Validation passes for SemanticValidation.
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app-test
Copy link
Copy Markdown

openapi-pipeline-app-test Bot commented Oct 27, 2020

Swagger Generation Artifacts

️️✔️~[Staging] ApiDocPreview succeeded [Detail]
 Please click here to preview with your @microsoft account. 
Posted by Swagger Pipeline | How to fix these errors?

openapi-sdkautomation-test Bot pushed a commit to test-repo-billy/azure-sdk-for-js that referenced this pull request Oct 27, 2020
Merge 2c250d014adc53f6b5da7c2deca0d60926acda6d into e21b960c504994bb94e16f96bf72d6da6ac06d25
openapi-sdkautomation-test Bot pushed a commit to test-repo-billy/azure-sdk-for-net that referenced this pull request Oct 27, 2020
Merge 2c250d014adc53f6b5da7c2deca0d60926acda6d into e21b960c504994bb94e16f96bf72d6da6ac06d25
@azure-sdk azure-sdk force-pushed the master branch 12 times, most recently from dfeeb02 to a7a0d0d Compare November 4, 2020 03:04
@azure-sdk azure-sdk force-pushed the master branch 10 times, most recently from 8b59dd7 to a2790be Compare November 11, 2020 03:02
@azure-sdk azure-sdk force-pushed the master branch 2 times, most recently from 0193d91 to 4d0807a Compare November 12, 2020 03:02
@azure-sdk azure-sdk force-pushed the master branch 12 times, most recently from 60b23e4 to ded7843 Compare November 19, 2020 03:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant