Skip to content

Conversation

@YuliiaKovalova
Copy link
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
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 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

@dotnet-policy-service
Copy link
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