Skip to content

Conversation

@TravisEz13
Copy link
Member

@TravisEz13 TravisEz13 commented Nov 10, 2025

This pull request updates the package validation workflow in the pipeline YAML files to improve the sequencing and location of the package validation stage. The main change is moving the validatePackages stage from the release pipeline to the package build pipeline, ensuring validation occurs earlier in the process.

Pipeline workflow changes:

  • Added the validatePackages stage to .pipelines/PowerShell-Packages-Official.yml, placing package name validation immediately after the upload stage in the package build pipeline.
  • Removed the validatePackages stage from .pipelines/PowerShell-Release-Official.yml, so package validation is no longer performed in the release pipeline.
  • Updated the dependencies in the release pipeline to remove references to the now-absent validatePackages stage.

@TravisEz13 TravisEz13 requested review from a team and jshigetomi as code owners November 10, 2025 21:30
Copilot AI review requested due to automatic review settings November 10, 2025 21:30
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 refactors the pipeline organization by moving the validatePackages stage from the release pipeline to the package pipeline, where package validation logically belongs. This improves separation of concerns by validating packages immediately after they are created and uploaded, rather than during the release process.

Key changes:

  • Removed validatePackages stage and its dependency from the release pipeline
  • Added validatePackages stage to the package pipeline after the upload stage
  • Fixed trailing whitespace on line 361

Reviewed Changes

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

File Description
.pipelines/PowerShell-Release-Official.yml Removed validatePackages stage and its dependency from UpdateChangeLog stage; fixed trailing whitespace
.pipelines/PowerShell-Packages-Official.yml Added validatePackages stage after upload stage to validate package names immediately after upload

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TravisEz13 TravisEz13 added CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log BackPort-7.4.x-Consider BackPort-7.5.x-Consider BackPort-7.6.x-Consider labels Nov 10, 2025
@TravisEz13 TravisEz13 merged commit 1934b83 into PowerShell:master Nov 10, 2025
45 of 46 checks passed
TravisEz13 added a commit to TravisEz13/PowerShell that referenced this pull request Nov 10, 2025
adityapatwardhan pushed a commit to adityapatwardhan/PowerShell that referenced this pull request Dec 2, 2025
SIRMARGIN pushed a commit to SIRMARGIN/PowerShell that referenced this pull request Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport-7.4.x-Migrated BackPort-7.5.x-Consider BackPort-7.6.x-Consider CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants