Content-Length: 412541 | pFad | https://github.com/Azure/azure-rest-api-specs/pull/32901

0C [DO NOT MERGE] Local changes for azure-cli by cxznmhdcxz · Pull Request #32901 · Azure/azure-rest-api-specs · GitHub
Skip to content

[DO NOT MERGE] Local changes for azure-cli #32901

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

[DO NOT MERGE] Local changes for azure-cli #32901

wants to merge 5 commits into from

Conversation

cxznmhdcxz
Copy link
Member

@cxznmhdcxz cxznmhdcxz commented Feb 28, 2025 •
edited
Loading

Local swagger changes for azure-cli dev
Including changes in #24526

@openapi-pipeline-app openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Feb 28, 2025 •
edited
Loading

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ This PR targets either the main branch of the public specs repo or the RPSaaSMaster branch of the private specs repo. These branches are not intended for iterative development. Therefore, you must acknowledge you understand that after this PR is merged, the APIs are considered shipped to Azure customers. Any further attempts at in-place modifications to the APIs will be subject to Azure's versioning and breaking change policies. Additionally, for control plane APIs, you must acknowledge that you are following all the best practices documented by ARM at aka.ms/armapibestpractices. If you do intend to release the APIs to your customers by merging this PR, add the PublishToCustomers label to your PR in acknowledgement of the above. Otherwise, retarget this PR onto a feature branch, i.e. with prefix release- (see aka.ms/azsdk/api-versions#release--branches).
  • ❌ This PR has at least one breaking change (label: BreakingChangeReviewRequired).
    To unblock this PR, follow the process at aka.ms/brch.
  • ❌ The required check named TypeSpec Validation has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide

@openapi-pipeline-app openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Feb 28, 2025 •
edited
Loading

PR validation pipeline restarted successfully. If there is ApiView generated, it will be updated in this comment.

@github-actions github-actions bot added the brownfield Brownfield services will soon be required to convert to TypeSpec. See https://aka.ms/azsdk/typespec. label Feb 28, 2025
@azure-sdk
Copy link
Collaborator

azure-sdk commented Feb 28, 2025 •
edited by github-actions bot
Loading

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Language API Review for Package
Swagger Microsoft.Compute
Swagger Microsoft.Network
Swagger Microsoft.Portal
Go sdk/resourcemanager/portal/armportal
JavaScript @azure/arm-portal
JavaScript @azure/arm-compute
Swagger Microsoft.Insights
Go sdk/resourcemanager/monitor/armmonitor
C# Azure.ResourceManager.Compute
JavaScript @azure/arm-network
C# Azure.ResourceManager.Network
Go sdk/resourcemanager/compute/armcompute
Python azure-mgmt-monitor_python.json
Python azure-mgmt-portal_python.json

This was referenced Feb 28, 2025
Draft
Draft
This was referenced Feb 28, 2025
Draft
Draft
Draft
Draft
Draft
Draft
Draft
@AzureRestAPISpecReview AzureRestAPISpecReview added BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required data-plane resource-manager TypeSpec Authored with TypeSpec labels Feb 28, 2025
This was referenced Feb 28, 2025
Draft
Draft
Draft
Draft
Draft
Draft
@cxznmhdcxz cxznmhdcxz force-pushed the local-changes-for-azure-cli branch from 8ed5eb5 to 586e3e5 Compare March 11, 2025 09:36
@cxznmhdcxz cxznmhdcxz force-pushed the local-changes-for-azure-cli branch from 586e3e5 to b6feaf5 Compare March 19, 2025 02:46
@cxznmhdcxz cxznmhdcxz force-pushed the local-changes-for-azure-cli branch from b6feaf5 to f006f15 Compare April 18, 2025 09:29
@azure-sdk azure-sdk mentioned this pull request Apr 18, 2025
Draft
This was referenced Apr 18, 2025
Draft
Draft
Draft
This was referenced Apr 18, 2025
Draft
Draft
@cxznmhdcxz cxznmhdcxz force-pushed the local-changes-for-azure-cli branch 2 times, most recently from 75218d7 to c1d901f Compare May 16, 2025 05:31
@cxznmhdcxz cxznmhdcxz force-pushed the local-changes-for-azure-cli branch from c1d901f to 187609f Compare May 30, 2025 03:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kamboj-prjwl kamboj-prjwl Awaiting requested review from kamboj-prjwl kamboj-prjwl will be requested when the pull request is marked ready for review kamboj-prjwl is a code owner

@dkulkarni-ms dkulkarni-ms Awaiting requested review from dkulkarni-ms dkulkarni-ms will be requested when the pull request is marked ready for review dkulkarni-ms is a code owner

@haagha haagha Awaiting requested review from haagha haagha will be requested when the pull request is marked ready for review haagha is a code owner

@MS-syh2qs MS-syh2qs Awaiting requested review from MS-syh2qs MS-syh2qs will be requested when the pull request is marked ready for review MS-syh2qs is a code owner

@grizzlytheodore grizzlytheodore Awaiting requested review from grizzlytheodore grizzlytheodore will be requested when the pull request is marked ready for review grizzlytheodore is a code owner

@mabhard mabhard Awaiting requested review from mabhard mabhard will be requested when the pull request is marked ready for review mabhard is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
BreakingChange-JavaScript-Sdk BreakingChange-Python-Sdk BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required brownfield Brownfield services will soon be required to convert to TypeSpec. See https://aka.ms/azsdk/typespec. data-plane Do Not Merge resource-manager RPaaS TypeSpec Authored with TypeSpec
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cxznmhdcxz @azure-sdk @AzureRestAPISpecReview








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: https://github.com/Azure/azure-rest-api-specs/pull/32901

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy