Skip to content

[Python] TypeSpec migration for azure-mgmt-resourcehealth#46323

Closed
ChenxiJiang333 wants to merge 3 commits into
Azure:mainfrom
ChenxiJiang333:sdk-azure-mgmt-resourcehealth
Closed

[Python] TypeSpec migration for azure-mgmt-resourcehealth#46323
ChenxiJiang333 wants to merge 3 commits into
Azure:mainfrom
ChenxiJiang333:sdk-azure-mgmt-resourcehealth

Conversation

@ChenxiJiang333

@ChenxiJiang333 ChenxiJiang333 commented Apr 15, 2026

Copy link
Copy Markdown
Member

[Python] TypeSpec migration for azure-mgmt-resourcehealth

Spec Source

Spec PR: Azure/azure-rest-api-specs#42359

Pre-migration swagger source: specification/resourcehealth/resource-manager@977f8ffa

Swagger API version: 2025-05-01 (default tag: package-2025-05-01)

Generated with apiVersion: 2025-05-01 (matched from swagger default tag)

Breaking Change Analysis

  • 0 unmitigated breaking changes (client rename already mitigated in spec PR)
  • 17 accepted breaking changes

Mitigations (already in spec PR)

Breaking Change Classification Mitigation
Client renamed ResourceHealthMgmtClientResourceHealthClient Guide #4 @@clientName(Microsoft.ResourceHealth, "ResourceHealthMgmtClient", "python")
ProxyResource duplicate name collision Compilation error @@clientName(ProxyResource, "ProxyResourceAutoGenerated", "python")

Note: The client.tsp needed using Microsoft.ResourceHealth; to resolve the unqualified ProxyResource reference — added locally for this generation.

Accepted Breaking Changes (will remain)

Guide #11 - Removal of multi-level flattened properties:

  • EmergingIssuesGetResult: refresh_timestamp, status_banners, status_active_events moved into nested properties object

Guide #8 - Removal of Pageable Models:

  • Events removed (pageable behavior preserved via ItemPaged)

Guide #7 - Removal of Unreferenced Models:

  • ImpactedResourceStatus removed
  • ReasonTypeValues removed

Guide #9 - Parameters Changed to Keyword-only:

  • expand param on AvailabilityStatuses, ChildAvailabilityStatuses, ChildResources operations
  • query_start_time param on Event, Events operations

Other:

  • Operations.list changed from async to sync

@ChenxiJiang333 ChenxiJiang333 force-pushed the sdk-azure-mgmt-resourcehealth branch from 1d7eff2 to d7e093a Compare April 15, 2026 08:30
@github-actions

Copy link
Copy Markdown
Contributor

Hi @ChenxiJiang333. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

@github-actions github-actions Bot added the no-recent-activity There has been no recent activity on this issue. label Jun 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hi @ChenxiJiang333. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing /reopen if you'd like to continue working on these changes. Please be sure to use the command to reopen or remove the no-recent-activity label; otherwise, this is likely to be closed again with the next cleanup pass.

@github-actions github-actions Bot closed this Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-recent-activity There has been no recent activity on this issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant