[DO NOT MERGE][Internal testing] test ARM spec change - Test 2#2934
[DO NOT MERGE][Internal testing] test ARM spec change - Test 2#2934konrad-jamrozik wants to merge 1 commit into
Conversation
Swagger Validation Report
|
| compared swaggers (via Oad v0.10.4)] | new version | base version |
|---|---|---|
| changes.json | 2023-07-01-preview(b08c4bc) | 2023-07-01-preview(main) |
| Rule | Message |
|---|---|
1049 - RemovedXmsEnum |
The new version is missing a 'x-ms-enum' found in the old version. New: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L303:9 Old: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L303:9 |
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️⚠️LintDiff: 0 Warnings warning [Detail]
| compared tags (via openapi-validator v2.1.3) | new version | base version |
|---|---|---|
| package-changes-2023-07-01-preview | package-changes-2023-07-01-preview(b08c4bc) | package-changes-2023-07-01-preview(main) |
The following errors/warnings exist before current PR submission:
| Rule | Message |
|---|---|
| Not using the common-types defined parameter 'subscriptionId'. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L49 |
|
| Not using the common-types defined parameter 'resourceGroupName'. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L49 |
|
| Not using the common-types defined parameter 'api-version'. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L49 |
|
| Not using the common-types defined parameter 'subscriptionId'. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L106 |
|
| Not using the common-types defined parameter 'resourceGroupName'. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L106 |
|
| Not using the common-types defined parameter 'api-version'. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L106 |
|
| Booleans properties are not descriptive in all cases and can make them to use, evaluate whether is makes sense to keep the property as boolean or turn it into an enum. Location: Microsoft.Resources/preview/2023-07-01-preview/changes.json#L261 |
️️✔️~[Staging] Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️ApiReadinessCheck succeeded [Detail] [Expand]
️⚠️~[Staging] ServiceAPIReadinessTest: 0 Warnings warning [Detail]
API Test is not triggered due to precheck failure. Check pipeline log for details.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️CadlAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️~[Staging] PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️~[Staging] SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️PR Summary succeeded [Detail] [Expand]
️️✔️CadlValidation succeeded [Detail] [Expand]
Validation passes for CadlValidation.
️️✔️TypeSpec Validation succeeded [Detail] [Expand]
Validation passes for TypeSpec Validation.
|
Hi, @konrad-jamrozik Thanks for your PR. I am workflow bot for review process. Here are some small tips. Any feedback about review process or workflow bot, pls contact swagger and tools team. vscswagger@microsoft.com |
Swagger Generation Artifacts
|
Generated ApiView
|
|
Please address or respond to feedback from the ARM API reviewer. When you are ready to continue the ARM API review, please remove the |
|
Hi @konrad-jamrozik! The automation detected breaking changes in this pull request. As a result, it added the |
This is a PR made by the Azure SDK Engineering System team.
ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can request review from an ARM API Review board.
ARM API changes review
ARMReviewlabel.ARMReviewlabel, if appropriate.If this happens, proceed according to guidance given in GitHub comments also added by the automation.
Breaking change review
If you have any breaking changes as defined in the Breaking Change Policy,
follow the process outlined in the High-level Breaking Change Process doc.
Getting help
and https://aka.ms/ci-fix.