Skip to content

Add signtype to parameters#12466

Merged
JanProvaznik merged 2 commits into
vs16.11from
dev/ykovalova/sign_variable_vs16.11
Sep 1, 2025
Merged

Add signtype to parameters#12466
JanProvaznik merged 2 commits into
vs16.11from
dev/ykovalova/sign_variable_vs16.11

Conversation

@YuliiaKovalova

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings August 31, 2025 06:20
@YuliiaKovalova YuliiaKovalova requested a review from a team as a code owner August 31, 2025 06:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 migrates the MSBuild Azure DevOps pipeline configuration from using a variable-based signing type to a parameter-based approach. This change provides better control over code signing configuration during pipeline execution.

Key changes:

  • Replace hardcoded SignType variable references with a configurable signTypeParameter
  • Add signTypeParameter to pipeline parameter definitions with appropriate defaults
  • Update task configurations to use the new parameter instead of the variable

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
azure-pipelines/.vsts-dotnet-exp-perf.yml Adds signTypeParameter with 'test' default and removes commented SignType variable
azure-pipelines/.vsts-dotnet-build-jobs.yml Updates MicroBuildSigningPlugin task to use parameter instead of variable
.vsts-dotnet.yml Adds signTypeParameter with 'real' default for production builds

Comment thread azure-pipelines/.vsts-dotnet-build-jobs.yml Outdated
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Hello! I noticed that you're targeting one of our servicing branches. Please consider updating the version.

@JanProvaznik JanProvaznik merged commit 8bab15e into vs16.11 Sep 1, 2025
5 checks passed
@JanProvaznik JanProvaznik deleted the dev/ykovalova/sign_variable_vs16.11 branch September 1, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants