Skip to content

[release/v7.6.2] Remove package verification from the notice pipeline#27425

Merged
daxian-dbw merged 1 commit into
PowerShell:release/v7.6.2from
daxian-dbw:backport/release/v7.6.2/27289-558c886d8
May 13, 2026
Merged

[release/v7.6.2] Remove package verification from the notice pipeline#27425
daxian-dbw merged 1 commit into
PowerShell:release/v7.6.2from
daxian-dbw:backport/release/v7.6.2/27289-558c886d8

Conversation

@daxian-dbw
Copy link
Copy Markdown
Member

Backport of #27289 to release/v7.6.2

Triggered by @daxian-dbw on behalf of @daxian-dbw

Original CL Label: CL-BuildPackaging

/cc @PowerShell/powershell-maintainers

Impact

REQUIRED: Choose either Tooling Impact or Customer Impact (or both). At least one checkbox must be selected.

Tooling Impact

  • Required tooling change
  • Optional tooling change (include reasoning)

Removes the now-redundant package verification step from the notice pipeline since the Release Prep workflow already handles updating cgmanifest.json.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Validated by the original PR. The package verification step is now handled by the Release Prep workflow, so removing it from the notice pipeline has no functional impact.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Removes a now-redundant step from the notice pipeline. The "Release Prep" workflow already handles updating cgmanifest.json, so removing package verification is a cleanup with no functional impact on the product.

Copilot AI review requested due to automatic review settings May 13, 2026 19:23
@daxian-dbw daxian-dbw requested review from a team and jshigetomi as code owners May 13, 2026 19:23
@daxian-dbw daxian-dbw added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label May 13, 2026
Copy link
Copy Markdown
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

Backport of #27289 to release/v7.6.2. Removes the now-redundant package license verification step from the notice pipeline since cgmanifest.json updates are handled by the Release Prep workflow.

Changes:

  • Removed SkipVerifyPackages parameter and the ClearlyDefined verification step from generateNotice.yml.
  • Removed the corresponding SkipVerifyPackages parameter and its passthrough from apiscan-gen-notice.yml.

Reviewed changes

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

File Description
.pipelines/templates/compliance/generateNotice.yml Drops the SkipVerifyPackages parameter and the ClearlyDefined -TestAndHarvest verification step.
.pipelines/apiscan-gen-notice.yml Removes the SkipVerifyPackages pipeline parameter and the value passed to the template.

@daxian-dbw daxian-dbw merged commit 712231c into PowerShell:release/v7.6.2 May 13, 2026
39 checks passed
@daxian-dbw daxian-dbw deleted the backport/release/v7.6.2/27289-558c886d8 branch May 13, 2026 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants