Skip to content

[release/v7.4.16] Remove package verification from the notice pipeline#27429

Merged
adityapatwardhan merged 1 commit into
PowerShell:release/v7.4.16from
adityapatwardhan:backport/release/v7.4.16/27289-558c886d8
May 15, 2026
Merged

[release/v7.4.16] Remove package verification from the notice pipeline#27429
adityapatwardhan merged 1 commit into
PowerShell:release/v7.4.16from
adityapatwardhan:backport/release/v7.4.16/27289-558c886d8

Conversation

@adityapatwardhan
Copy link
Copy Markdown
Member

Backport of #27289 to release/v7.4.16

Triggered by @adityapatwardhan 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 unnecessary verification step from CI pipeline. Ensures pipeline is up-to-date with current release process.

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Validated pipeline changes by running the updated workflow. No failures observed and cgmanifest updates are handled by Release Prep workflow.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

This change only removes redundant package verification from the notice pipeline, which is now handled elsewhere. No impact on product code or customer scenarios.

Copilot AI review requested due to automatic review settings May 13, 2026 20:06
@adityapatwardhan adityapatwardhan requested a review from a team as a code owner May 13, 2026 20:06
@adityapatwardhan adityapatwardhan 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

This PR backports the change from #27289 to release/v7.4.16, removing the package/license verification step from the NOTICE generation pipeline now that the Release Prep workflow handles cgmanifest.json updates.

Changes:

  • Removed the SkipVerifyPackages parameter from the NOTICE-generation template and its callers.
  • Dropped the ClearlyDefined -TestAndHarvest verification step from the NOTICE pipeline job.

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 Removes the optional package verification parameter and the ClearlyDefined verification step from the NOTICE generation job.
.pipelines/apiscan-gen-notice.yml Removes the unused SkipVerifyPackages parameter and stops passing it into the NOTICE template.

@adityapatwardhan adityapatwardhan merged commit 3dc4808 into PowerShell:release/v7.4.16 May 15, 2026
40 checks passed
@adityapatwardhan adityapatwardhan deleted the backport/release/v7.4.16/27289-558c886d8 branch May 15, 2026 17:43
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.

4 participants