Skip to content

Use azpysdk Breaking Check in CI#44215

Merged
scbedd merged 3 commits intoAzure:mainfrom
JennyPng:jennypng-cut-breaking
Dec 18, 2025
Merged

Use azpysdk Breaking Check in CI#44215
scbedd merged 3 commits intoAzure:mainfrom
JennyPng:jennypng-cut-breaking

Conversation

@JennyPng
Copy link
Member

@JennyPng JennyPng commented Dec 2, 2025

@scbedd
Copy link
Member

scbedd commented Dec 17, 2025

Lets run an invocation of ml and storage and ensure that it's matching.

Copy link
Member

@scbedd scbedd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull off draft, attempt verify it matches for storage and ml, verify it then I think we're g2g on verification.

@JennyPng JennyPng marked this pull request as ready for review December 17, 2025 19:18
Copilot AI review requested due to automatic review settings December 17, 2025 19:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request migrates the breaking changes CI check from the legacy dispatch_tox.py script to the newer dispatch_checks.py script, which uses the azpysdk breaking check tool instead of tox. This aligns the breaking changes pipeline with the migration effort tracked in issue #42601.

Key Changes

  • Switches from tox-based breaking changes detection to azpysdk-based breaking changes detection
  • Updates script path and argument names to match the new dispatch framework
  • Adds explicit environment variable configuration for proper isolation

Copy link
Member

@scbedd scbedd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JennyPng you're done legit as much verification as we can ask for. Any issues that come back will just be fixed in main. Thank you for your patience and all the work on this!

@scbedd
Copy link
Member

scbedd commented Dec 18, 2025

/check-enforcer override

@scbedd scbedd merged commit 5c989a4 into Azure:main Dec 18, 2025
31 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants