Skip to content

Change create release plan method to hide work item details#11931

Merged
smw-ms merged 8 commits into
Azure:mainfrom
smw-ms:releaseplan-abstraction
Sep 15, 2025
Merged

Change create release plan method to hide work item details#11931
smw-ms merged 8 commits into
Azure:mainfrom
smw-ms:releaseplan-abstraction

Conversation

@smw-ms

@smw-ms smw-ms commented Sep 3, 2025

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings September 3, 2025 00:01
@github-actions github-actions Bot added the azsdk-cli Issues related to Azure/azure-sdk-tools::tools/azsdk-cli label Sep 3, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 changes the create release plan method to hide work item details by returning a simplified release plan object instead of the raw work item details. The changes extract specific fields from the work item and populate a ReleasePlanDetails object for cleaner output.

  • Updated the CreateReleasePlan method to extract work item ID and custom fields into a ReleasePlanDetails object
  • Modified test assertions to verify the new return type instead of the work item type
  • Added documentation note to direct users to the Release Planner Tool for manual updates

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
ReleasePlanTool.cs Modified to extract work item fields into ReleasePlanDetails object instead of returning raw work item
ReleasePlanToolTests.cs Updated test assertions to check for ReleasePlanDetails type instead of WorkItem type
create-release-plan.instructions.md Added note directing users to Release Planner Tool for manual updates

@azure-sdk

Copy link
Copy Markdown
Collaborator

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

@smw-ms smw-ms linked an issue Sep 3, 2025 that may be closed by this pull request

@praveenkuttappan praveenkuttappan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks fine to me. Please sync instruction file to all repos.

@smw-ms

smw-ms commented Sep 5, 2025

Copy link
Copy Markdown
Member Author

/check-enforcer override

@maririos maririos left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you provide screenshots of this on the works? how is the output now?
For example, before this change when the user requested the creation of the release plan, the answer from the agent was:
image

Comment thread eng/common/instructions/azsdk-tools/create-release-plan.instructions.md Outdated
Comment thread tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/ReleasePlan/ReleasePlanTool.cs Outdated
Comment thread eng/common/instructions/azsdk-tools/create-release-plan.instructions.md Outdated
…ctions.md

Co-authored-by: Mariana Rios Flores <mariari@microsoft.com>
@azure-sdk

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Collaborator

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

Comment thread eng/common/instructions/azsdk-tools/create-release-plan.instructions.md Outdated
Comment thread tools/azsdk-cli/Azure.Sdk.Tools.Cli/Models/ReleasePlanDetails.cs Outdated
…ctions.md

Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
@azure-sdk

Copy link
Copy Markdown
Collaborator

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

@smw-ms

smw-ms commented Sep 10, 2025

Copy link
Copy Markdown
Member Author

Could you provide screenshots of this on the works? how is the output now? For example, before this change when the user requested the creation of the release plan, the answer from the agent was: image

#11656 (comment)
#11656 (comment)

@azure-sdk

Copy link
Copy Markdown
Collaborator

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

Comment thread eng/common/instructions/azsdk-tools/create-release-plan.instructions.md Outdated
@azure-sdk

Copy link
Copy Markdown
Collaborator

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

@smw-ms

smw-ms commented Sep 15, 2025

Copy link
Copy Markdown
Member Author

/check-enforcer override

@smw-ms
smw-ms merged commit 7a8f637 into Azure:main Sep 15, 2025
16 of 17 checks passed
smw-ms added a commit to Azure/azure-sdk-for-rust that referenced this pull request Sep 15, 2025
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#11931 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: Summer Warren <summerwarren@microsoft.com>
Co-authored-by: Mariana Rios Flores <mariari@microsoft.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
smw-ms added a commit to Azure/azure-sdk-for-js that referenced this pull request Sep 15, 2025
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#11931 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: Summer Warren <summerwarren@microsoft.com>
Co-authored-by: Mariana Rios Flores <mariari@microsoft.com>
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
@smw-ms
smw-ms deleted the releaseplan-abstraction branch October 13, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Release plan should alway be communicated in terms of the PowerApp

6 participants