Skip to content

Conversation

@raych1
Copy link
Member

@raych1 raych1 commented Nov 19, 2025

Make this input switch from artifacts to packageInfo since both of the 'API Review' pipeline and 'Validate Package` pipeline had supported this.

Test runs:
patch-release
template-batch-release

Copilot AI review requested due to automatic review settings November 19, 2025 19:04
@raych1 raych1 self-assigned this Nov 19, 2025
Copilot finished reviewing on behalf of raych1 November 19, 2025 19:08
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 modernizes the pipeline configuration by switching from passing Artifacts parameter to PackageInfoFiles parameter when calling the API review and package validation templates. Both templates now support the PackageInfoFiles parameter, which provides a more direct way to specify package information files rather than constructing them from artifact names.

  • Replaces Artifacts parameter with PackageInfoFiles in template calls to create-apireview.yml and validate-all-packages.yml
  • Constructs explicit file paths for each package info JSON file using ${{ each artifact }} loops
  • Maintains backward compatibility as the underlying templates still support both parameters with proper fallback logic

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
eng/pipelines/templates/stages/archetype-java-release-patch.yml Updates create-apireview template call to use PackageInfoFiles instead of Artifacts parameter
eng/pipelines/templates/stages/archetype-java-release-batch.yml Updates create-apireview template call to use PackageInfoFiles instead of Artifacts parameter
eng/pipelines/templates/jobs/ci.yml Updates both create-apireview and validate-all-packages template calls to use PackageInfoFiles instead of Artifacts parameter

@raych1 raych1 requested a review from a team as a code owner November 19, 2025 21:57
@raych1 raych1 moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🍔🌭 Nov 20, 2025
@raych1
Copy link
Member Author

raych1 commented Nov 21, 2025

This PR has to wait after the PR is merged.

@raych1 raych1 merged commit 433c001 into main Nov 21, 2025
13 checks passed
@raych1 raych1 deleted the users/raych1/update-pipeline-script-for-v2 branch November 21, 2025 19:32
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🍔🌭 Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎊 Closed

Development

Successfully merging this pull request may close these issues.

4 participants