Define MsbuildSigningArguments Based on Public vs Internal (#4290)

* Remove explicit signing argument set

* Define MsbuildSigningArguments based on conditional

* be more explicit

* remove unneeded group

eng/pipelines/jobs/windows-build-PR.yml

@@ -30,15 +30,16 @@ jobs:
           -c $(_BuildConfig)
           /p:TargetArchitecture=${{ parameters.targetArchitecture }}
           /p:SkipTests=${{ parameters.skipTests }}
-      - name: MsbuildSigningArguments
-        value: /p:DotNetSignType=$(SignType)
       - name: TargetArchitecture
         value: ${{ parameters.targetArchitecture }}
       - ${{ if eq(variables['System.TeamProject'], 'public') }}:
+        - name: MsbuildSigningArguments
+          value: /p:DotNetSignType=Test
         - name: _InternalRuntimeDownloadArgs
           value: ''
       - ${{ if ne(variables['System.TeamProject'], 'public') }}:
-        - group: DotNet-MSRC-Storage
+        - name: MsbuildSigningArguments
+          value: /p:DotNetSignType=Real
         - name: _InternalRuntimeDownloadArgs
           value: >-
             /p:DotNetRuntimeSourceFeed=https://dotnetclimsrc.blob.core.windows.net/dotnet
@@ -107,4 +108,4 @@ jobs:
         PathtoPublish: '$(Build.StagingDirectory)/BuildLogs'
         ArtifactName: Logs-${{ parameters.name }}-$(_BuildConfig)
       continueOnError: true
-      condition: succeededOrFailed()
+      condition: succeededOrFailed()