-
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
Add description to Key Vault 7.3 KeyReleasePolicy #19613
Conversation
Hi, @chlowell 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. [email protected] |
Swagger Validation Report
|
compared swaggers (via Oad v0.9.6)] | new version | base version |
---|---|---|
keys.json | 7.3(3c049cf) | 7.3(main) |
️️✔️
Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️
CredScan succeeded [Detail] [Expand]
There is no credential detected.
️️✔️
LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
compared tags (via openapi-validator v1.13.0) | new version | base version |
---|---|---|
package-7.3 | package-7.3(3c049cf) | package-7.3(main) |
️️✔️
Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️
ApiReadinessCheck succeeded [Detail] [Expand]
️️✔️
ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️
SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️
PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️⚠️
SDK Track2 Validation: 0 Warnings warning [Detail]
- The following tags are being changed in this PR
|:speech_balloon: |"readme":"keyvault/data-plane/readme.md",
"tag":"package-7.3",
"details":"> Installing AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0 -> 1.11.0)"|
|:speech_balloon: |"readme":"keyvault/data-plane/readme.md",
"tag":"package-7.3",
"details":"> Installed AutoRest extension '@microsoft.azure/openapi-validator' (1.11.0->1.11.0)"|
|:speech_balloon: |"readme":"keyvault/data-plane/readme.md",
"tag":"package-7.3",
"details":"> Installing AutoRest extension '@autorest/modelerfour' (4.21.4 -> 4.21.4)"|
|:speech_balloon: |"readme":"keyvault/data-plane/readme.md",
"tag":"package-7.3",
"details":"> Installed AutoRest extension '@autorest/modelerfour' (4.21.4->4.21.4)"|
|:speech_balloon: |"readme":"keyvault/data-plane/readme.md",
"tag":"package-7.3",
"details":"Autorest completed in 13.03s. 0 files generated."|
The following errors/warnings exist before current PR submission:
Only 30 items are listed, please refer to log for more details.
Rule | Message |
---|---|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.0.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.1.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.2.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.3.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.4.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.5.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.6.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.7.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.8.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.9.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.10.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.11.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.12.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.13.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.14.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.15.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.16.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.17.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.18.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.19.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.20.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.21.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.22.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.23.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.24.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.25.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.26.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.27.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.28.suppress which is deprecated and will be removed in the future." |
|
"readme":"keyvault/data-plane/readme.md", "tag":"package-7.3", "details":"Using directive.29.suppress which is deprecated and will be removed in the future." |
️️✔️
PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️
SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️
Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️
CadlValidation succeeded [Detail] [Expand]
Validation passes for CadlValidation.
Swagger pipeline restarted successfully, please wait for status update in this comment. |
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.
Looks good! 👍
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.
Grammatically correct to boot i.e., not ending the sentence in a preposition!
No API change here, just adding a description to one of the models.