From e0a72d067b625e62360a7adf91c92334f6a540e0 Mon Sep 17 00:00:00 2001
From: David Barbet <dabarbet@microsoft.com>
Date: Wed, 22 Jan 2025 12:56:53 -0800
Subject: [PATCH] Revert "Change PR Val builds to be unsigned (#76358)"

This reverts commit 2eca5b28c9dd60e09fe4ebdc8544b9efce18d713, reversing
changes made to fe3a243bd58966b3e683c9d038321ddb498ff043.
---
 azure-pipelines-pr-validation.yml |  2 --
 eng/Signing.props                 | 15 ---------------
 2 files changed, 17 deletions(-)

diff --git a/azure-pipelines-pr-validation.yml b/azure-pipelines-pr-validation.yml
index 2b8394f1b5d82..5399ec92521dc 100644
--- a/azure-pipelines-pr-validation.yml
+++ b/azure-pipelines-pr-validation.yml
@@ -33,8 +33,6 @@ variables:
     value: false
   - name: Codeql.SkipTaskAutoInjection
     value: true
-  - name: SignType
-    value: ''
 
   # To retrieve OptProf data we need to authenticate to the VS drop storage.
   # If the pipeline is running in DevDiv, the account has access to the VS drop storage.
diff --git a/eng/Signing.props b/eng/Signing.props
index 0bda27d7aa9f1..f1d94e766f317 100644
--- a/eng/Signing.props
+++ b/eng/Signing.props
@@ -61,19 +61,4 @@
     <FileSignInfo Include="Microsoft.VisualStudio.Threading.dll" CertificateName="MicrosoftSHA2" />
     <FileSignInfo Include="StreamJsonRpc.dll" CertificateName="MicrosoftSHA2" />
   </ItemGroup>
-
-  <!--
-    We do not intend to ship the artifacts produced for PR validation build so we can skip signing the artifacts.
-   -->
-  <Choose>
-    <When Condition="'$(PreReleaseVersionLabel)' == 'pr-validation' And '$(SignType)' == ''">
-      <ItemGroup>
-        <ItemsToSign Remove="@(ItemsToSign)" />
-      </ItemGroup>
-
-      <PropertyGroup>
-        <AllowEmptySignList>true</AllowEmptySignList>
-      </PropertyGroup>
-    </When>
-  </Choose>
 </Project>