Skip to content

Add a dry run of package release cli command#13134

Merged
jeo02 merged 14 commits intoAzure:mainfrom
jeo02:dry-run=package-release
Dec 10, 2025
Merged

Add a dry run of package release cli command#13134
jeo02 merged 14 commits intoAzure:mainfrom
jeo02:dry-run=package-release

Conversation

@jeo02
Copy link
Member

@jeo02 jeo02 commented Dec 2, 2025

@jeo02 jeo02 requested a review from a team as a code owner December 2, 2025 20:16
Copilot AI review requested due to automatic review settings December 2, 2025 20:16
@github-actions github-actions bot added the azsdk-cli Issues related to Azure/azure-sdk-tools::tools/azsdk-cli label Dec 2, 2025
jeo02
jeo02 commented Dec 2, 2025
View reviewed changes
@jeo02 jeo02 self-assigned this Dec 2, 2025
@jeo02 jeo02 requested a review from benbp December 2, 2025 20:19
Copilot AI reviewed Dec 2, 2025
View reviewed changes
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 PR replaces the standalone azsdk pkg release-readiness command with a --dry-run option for the azsdk pkg release command, consolidating related functionality into a single command interface.

Key changes:

  • Added a --dry-run boolean option to the release command that performs readiness checks without triggering the release pipeline
  • Updated the ReleasePackageAsync method signature to accept an optional dryRun parameter
  • Removed ReleaseReadinessTool from the SharedOptions tool registry

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/Package/SdkReleaseTool.cs Added --dry-run option and corresponding logic to perform readiness checks without triggering the release pipeline
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Commands/SharedOptions.cs Removed ReleaseReadinessTool from the list of registered command tools

@jeo02 jeo02 requested a review from MrJustinB as a code owner December 3, 2025 22:50
@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 3, 2025

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 3, 2025

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

benbp
benbp reviewed Dec 3, 2025
View reviewed changes
benbp
benbp reviewed Dec 3, 2025
View reviewed changes
@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 4, 2025

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 4, 2025

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

This was referenced Dec 4, 2025
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 4, 2025

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@azure-sdk
Copy link
Collaborator

azure-sdk commented Dec 9, 2025

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Dec 10, 2025
@azure-sdk @jeo02
Sync eng/common directory with azure-sdk-tools for PR 13134 (#36805)
d3a66f1 
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#13134 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Dec 10, 2025
@azure-sdk @jeo02
Sync eng/common directory with azure-sdk-tools for PR 13134 (#3398)
95075b4 
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#13134 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
@jeo02 jeo02 enabled auto-merge (squash) December 10, 2025 19:32
@jeo02 jeo02 merged commit 73953a5 into Azure:main Dec 10, 2025
16 checks passed
JiaqiZhang-Dev pushed a commit that referenced this pull request Dec 22, 2025
@jeo02
Add a dry run of package release cli command (#13134)
cf6f0bb 
* Add a dry run of package release cli command

* Update tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/Package/SdkReleaseTool.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* test with commented out part for question

* update description + nit

* Remove ReleaseReadinessTool and move it into SdkReleaseTool

* instruction fix

* fix tests

* shared otpions

* check-ready option

* rename test

* description + instructions

* fix

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benbp benbp benbp left review comments

Copilot code review Copilot Copilot left review comments

@praveenkuttappan praveenkuttappan praveenkuttappan approved these changes

@weshaggard weshaggard Awaiting requested review from weshaggard

@maririos maririos Awaiting requested review from maririos maririos is a code owner

@MrJustinB MrJustinB Awaiting requested review from MrJustinB MrJustinB is a code owner

Assignees

@jeo02 jeo02

Labels

azsdk-cli Issues related to Azure/azure-sdk-tools::tools/azsdk-cli

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Az SDK MCP server: Add a dry run of package release cli command

5 participants

@jeo02 @azure-sdk @benbp @praveenkuttappan