Skip to content

Added optional artifact list to filter the package info to be returned#13202

Merged
raych1 merged 5 commits intomainfrom
users/raych1/update-save-package-info-script
Dec 5, 2025
Merged

Added optional artifact list to filter the package info to be returned#13202
raych1 merged 5 commits intomainfrom
users/raych1/update-save-package-info-script

Conversation

@raych1
Copy link
Member

@raych1 raych1 commented Dec 4, 2025

Allow the script to create package info for the specified artifact list.

Copilot AI review requested due to automatic review settings December 4, 2025 18:06
@raych1 raych1 requested a review from a team as a code owner December 4, 2025 18:06
@raych1 raych1 self-assigned this Dec 4, 2025
@raych1 raych1 requested a review from weshaggard December 4, 2025 18:06
Copilot AI reviewed Dec 4, 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 adds optional artifact filtering capability to the Save-Package-Properties.ps1 script, allowing users to limit package property generation to only specified artifacts.

Key Changes:

  • Added new artifactList parameter to filter package properties by artifact names
  • Implemented validation and filtering logic with case-insensitive matching using HashSet for performance

@raych1
Apply suggestions from code review
74af4d2 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@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

@raych1 raych1 moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 📆🎇 Dec 4, 2025
@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
This was referenced Dec 4, 2025
Merged
Merged
azure-sdk added a commit to Azure/azure-sdk-for-js that referenced this pull request Dec 5, 2025
@azure-sdk @raych1
Sync eng/common directory with azure-sdk-tools for PR 13202 (#36808)
3a4669d 
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#13202 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: ray chen <raychen@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
azure-sdk added a commit to Azure/azure-sdk-for-rust that referenced this pull request Dec 5, 2025
@azure-sdk @raych1
Sync eng/common directory with azure-sdk-tools for PR 13202 (#3399)
48ff829 
Sync eng/common directory with azure-sdk-tools for PR
Azure/azure-sdk-tools#13202 See [eng/common
workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow)

---------

Co-authored-by: ray chen <raychen@microsoft.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@raych1 raych1 merged commit 41d147f into main Dec 5, 2025
10 checks passed
@raych1 raych1 deleted the users/raych1/update-save-package-info-script branch December 5, 2025 22:58
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 📆🎇 Dec 9, 2025
@raych1 raych1 linked an issue Dec 9, 2025 that may be closed by this pull request
Support package info filtering by an optional artifact list #13284
Closed
JiaqiZhang-Dev pushed a commit that referenced this pull request Dec 22, 2025
@raych1
Added optional artifact list to filter the package info to be returned (
f2d66ed 
#13202)

* Added optional artifact list to filter the package info to be returned

* Apply suggestions from code review

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

* return full package info if the input artifact list is empty

* Fixed hashset issue

* Added artifacts parameter

---------

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

Copilot code review Copilot Copilot left review comments

@weshaggard weshaggard weshaggard approved these changes

Assignees

@raych1 raych1

Labels

None yet

Projects

Azure SDK EngSys 📆🎇
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Support package info filtering by an optional artifact list

3 participants

@raych1 @azure-sdk @weshaggard