Skip to content

[release/v7.5] Remove unused runCodesignValidationInjection variable from pipeline templates#26825

Merged
daxian-dbw merged 1 commit intoPowerShell:release/v7.5from
daxian-dbw:backport/release/v7.5/26412-1241ad279
Feb 14, 2026
Merged

[release/v7.5] Remove unused runCodesignValidationInjection variable from pipeline templates#26825
daxian-dbw merged 1 commit intoPowerShell:release/v7.5from
daxian-dbw:backport/release/v7.5/26412-1241ad279

Conversation

@daxian-dbw
Copy link
Member

Backport of #26412 to release/v7.5

Triggered by @daxian-dbw on behalf of @app/copilot-swe-agent

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 unused runCodesignValidationInjection variable from 14 pipeline templates

Customer Impact

  • Customer reported
  • Found internally

Regression

REQUIRED: Check exactly one box.

  • Yes
  • No

This is not a regression.

Testing

Successfully tested in 7.6 release. Removes dead code with no impact on pipeline functionality.

Risk

REQUIRED: Check exactly one box.

  • High
  • Medium
  • Low

Removes unused variable declarations from pipeline templates. Code cleanup with no functional impact. Successfully backported to 7.6 branch.

Merge Conflicts

Conflict in release-shared.yml resolved - context differences in variable definitions

…emplates (PowerShell#26412)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: TravisEz13 <[email protected]>
Co-authored-by: Travis Plunk (HE/HIM) <[email protected]>
@daxian-dbw daxian-dbw requested a review from a team as a code owner February 14, 2026 00:43
Copilot AI review requested due to automatic review settings February 14, 2026 00:43
@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 Feb 14, 2026
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 pull request backports the removal of the unused runCodesignValidationInjection variable from the v7.5 release branch. The variable was defined with value: false across 14 pipeline template files but was never referenced in any pipeline logic, making it dead code that can be safely removed.

Changes:

  • Removed unused runCodesignValidationInjection variable declarations from 14 Azure Pipeline YAML templates
  • Resolved merge conflict in release-shared.yml during backport

Reviewed changes

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

Show a summary per file
File Description
.pipelines/templates/windows-hosted-build.yml Removed unused variable definition
.pipelines/templates/variable/release-shared.yml Removed unused variable definition with merge conflict resolution
.pipelines/templates/uploadToAzure.yml Removed unused variable definition
.pipelines/templates/testartifacts.yml Removed unused variable definition from both Windows and non-Windows jobs
.pipelines/templates/release-upload-buildinfo.yml Removed unused variable definition
.pipelines/templates/release-symbols.yml Removed unused variable definition
.pipelines/templates/packaging/windows/package.yml Removed unused variable definition
.pipelines/templates/nupkg.yml Removed unused variable definition
.pipelines/templates/mac.yml Removed unused variable definition
.pipelines/templates/mac-package-build.yml Removed unused variable definition
.pipelines/templates/linux.yml Removed unused variable definition from both Linux and Windows signing jobs
.pipelines/templates/linux-package-build.yml Removed unused variable definition
.pipelines/templates/compliance/generateNotice.yml Removed unused variable definition
.pipelines/templates/compliance/apiscan.yml Removed unused variable definition

@daxian-dbw daxian-dbw merged commit 5168d15 into PowerShell:release/v7.5 Feb 14, 2026
41 checks passed
@daxian-dbw daxian-dbw deleted the backport/release/v7.5/26412-1241ad279 branch February 14, 2026 00:54
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