Skip to content

Msyyc patch 3 #3333

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

Closed
wants to merge 4 commits into from
Closed

Msyyc patch 3 #3333

wants to merge 4 commits into from

Conversation

msyyc
Copy link

@msyyc msyyc commented Apr 28, 2024

No description provided.

Copy link

openapi-pipeline-app-test bot commented Apr 28, 2024

Next Steps to merge

Next steps that must be taken to merge this PR:
  • ❌ Your PR has breaking changes in the generated SDK for Python (label: BreakingChange-Python-Sdk). Refer to step 3 in the PR workflow diagram (even if your PR is for data plane, not ARM).
  • ❌ The required check named Automated merging requirements met has failed. This is the final check that must pass. Refer to the check in the PR's 'Checks' tab for details on how to fix it. In addition, refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

Copy link

openapi-pipeline-app-test bot commented Apr 28, 2024

Swagger Generation Artifacts

️️✔️[ignore this check status; not blocking PR merge] ApiDocPreview succeeded [Detail] [Expand]
️⚠️ azure-sdk-for-python warning [Detail]
  • ⚠️Warning in generating from 62e5ee267a4e0145c098aa0f1cb56e1d71953243. SDK Automation 14.0.0
    command	sh scripts/automation_init.sh ../azure-sdk-for-python_tmp/initInput.json ../azure-sdk-for-python_tmp/initOutput.json
    cmderr	[automation_init.sh] WARNING: Skipping azure-nspkg as it is not installed.
    command	sh scripts/automation_generate.sh ../azure-sdk-for-python_tmp/generateInput.json ../azure-sdk-for-python_tmp/generateOutput.json
  • ️✔️azure-mgmt-deviceregistry [Preview SDK Changes] Breaking Change Detected
    • azure-mgmt-deviceregistry-1.0.0b1.zip
    • azure_mgmt_deviceregistry-1.0.0b1-py3-none-any.whl
    info	[Changelog] ### Breaking Changes
    info	[Changelog]
    info	[Changelog]   - Model Asset no longer has parameter name
    info	[Changelog]   - Model AssetEndpointProfile no longer has parameter name
    info	[Changelog]   - Parameter id of model Asset is now required
    info	[Changelog]   - Parameter id of model AssetEndpointProfile is now required
    info	[Changelog]   - Parameter operations of model OperationStatusResult is now required
    info	[Changelog]   - Parameter type of model Asset is now required
    info	[Changelog]   - Parameter type of model AssetEndpointProfile is now required
    Present SDK breaking changes suppressions
    No suppression file added.
    Absent SDK breaking changes suppressions
    - Model Asset no longer has parameter name
    - Model AssetEndpointProfile no longer has parameter name
    - Parameter id of model Asset is now required
    - Parameter id of model AssetEndpointProfile is now required
    - Parameter operations of model OperationStatusResult is now required
    - Parameter type of model Asset is now required
    - Parameter type of model AssetEndpointProfile is now required
    Parse Suppression File Errors
    No suppression file added.
    Please refer to https://aka.ms/azsdk/sdk-suppression for more information.
Posted by Swagger Pipeline | How to fix these errors?

Copy link

openapi-pipeline-app-test bot commented Apr 28, 2024

Generated ApiView

Language Package Name ApiView Link
TypeSpec DeviceRegistry.Management https://apiviewstagingtest.com/Assemblies/Review/8c8020c228c948c5bb66512e800753e8?revisionId=392760cf1aab488980864b33cbfa1144

Copy link

openapi-pipeline-app-test bot commented Apr 28, 2024

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
️️✔️[ignore this check status; not blocking PR merge] Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️SwaggerAPIView 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.
️️✔️[ignore this check status; not blocking PR merge] SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️PR Summary succeeded [Detail] [Expand]
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

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

Successfully merging this pull request may close these issues.

3 participants