Skip to content

Update readme.python.md#26711

Merged
ArthurMa1978 merged 6 commits into
Azure:mainfrom
ChenxiJiang333:ChenxiJiang333-patch-4
Nov 27, 2023
Merged

Update readme.python.md#26711
ArthurMa1978 merged 6 commits into
Azure:mainfrom
ChenxiJiang333:ChenxiJiang333-patch-4

Conversation

@ChenxiJiang333
Copy link
Copy Markdown
Member

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Nov 15, 2023

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named SDK azure-sdk-for-go has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it. In addition, refer to step 3 in the PR workflow diagram (even if your PR is for data plane, not ARM).

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Nov 15, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️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 v2.1.6) new version base version
package-preview-2023-02-15-only package-preview-2023-02-15-only(0e0b9b7) default(main)
️⚠️Avocado: 1 Warnings warning [Detail]
Rule Message
⚠️ MULTIPLE_API_VERSION The default tag contains multiple API versions swaggers.
readme: specification/security/resource-manager/readme.md
tag: specification/security/resource-manager/readme.md#tag-package-composite-v3
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView 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.
️️✔️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).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Nov 15, 2023

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
️❌SDK Breaking Change Tracking failed [Detail]

Only 0 items are rendered, please refer to log for more details.

️️✔️ azure-sdk-for-net-track2 succeeded [Detail] [Expand]

Only 0 items are rendered, please refer to log for more details.

️❌ azure-sdk-for-go failed [Detail]

Only 0 items are rendered, please refer to log for more details.

️⚠️ azure-sdk-for-python-track2 warning [Detail]

Only 0 items are rendered, please refer to log for more details.

️️✔️ azure-sdk-for-js succeeded [Detail] [Expand]

Only 0 items are rendered, please refer to log for more details.

️⚠️ azure-sdk-for-java warning [Detail]

Only 0 items are rendered, please refer to log for more details.

️⚠️ azure-resource-manager-schemas warning [Detail]

Only 0 items are rendered, please refer to log for more details.

️❌ azure-powershell failed [Detail]

Only 0 items are rendered, please refer to log for more details.

Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link
Copy Markdown

openapi-pipeline-app Bot commented Nov 15, 2023

Generated ApiView

Language Package Name ApiView Link
Java azure-resourcemanager-security https://apiview.dev/Assemblies/Review/9c5268287768433c9e58bab75cc84488
.Net Azure.ResourceManager.SecurityCenter https://apiview.dev/Assemblies/Review/774f3e585ff745958fdf951968858f7c
JavaScript @azure/arm-security https://apiview.dev/Assemblies/Review/fbed12cb608f4cc795dff21ea026d5fa

@openapi-pipeline-app
Copy link
Copy Markdown

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@openapi-pipeline-app
Copy link
Copy Markdown

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@openapi-pipeline-app
Copy link
Copy Markdown

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@openapi-pipeline-app
Copy link
Copy Markdown

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@msyyc msyyc added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review Approved-SdkBreakingChange-JavaScript labels Nov 17, 2023
@openapi-workflow-bot openapi-workflow-bot Bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Nov 17, 2023
@msyyc msyyc enabled auto-merge (squash) November 17, 2023 03:09
@salambashir
Copy link
Copy Markdown
Member

Hi @msyyc , Thanks for approving.
I see you enabled auto-merge, but the sdk for go check is failing.
We are only interested in updating the python sdk. How can we address this from here?
Thanks for your help :)

@msyyc
Copy link
Copy Markdown
Member

msyyc commented Nov 23, 2023

@raych1 Please help fix Go failure.

@raych1
Copy link
Copy Markdown
Member

raych1 commented Nov 27, 2023

There is a known autorest issue that caused the Go generation failure.
@lirenhe , please help force merge this PR.

@lirenhe
Copy link
Copy Markdown
Member

lirenhe commented Nov 27, 2023

There is a known autorest issue that caused the Go generation failure. @lirenhe , please help force merge this PR.

I don't have the permission for force merge now.

@ArthurMa1978 ArthurMa1978 merged commit 739f4f5 into Azure:main Nov 27, 2023
@ChenxiJiang333 ChenxiJiang333 deleted the ChenxiJiang333-patch-4 branch January 19, 2024 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved-SdkBreakingChange-JavaScript ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review CI-BreakingChange-JavaScript ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants