fix: correct CI to run if no approval is required

.azure/pipelines/templates/build.yaml

@@ -51,20 +51,23 @@ parameters:
     displayName: Run CodeQL3000 tasks
     type: boolean
 
+
 jobs:
 
 # Approval needed for publishing to nuget.org
-- ${{ if and(eq(parameters.codesign, true), eq(parameters.publish_nuget, true)) }}:
-  - job: PreDeploymentApprovalJob
-    displayName: Pre-Deployment Approval
-    condition: succeeded()
-    timeoutInMinutes: 2880
+- job: PreDeploymentApprovalJob
+  displayName: Pre-Deployment Approval
+  timeoutInMinutes: 2880
+  ${{ if and(eq(parameters.codesign, true), eq(parameters.publish_nuget, true)) }}:
     pool: server
-    steps:
-    - task: ManualValidation@1
-      inputs:
-        notifyUsers: ${{ variables.notifyUsers }}
-        approvers: ${{ variables.approvers }}
+  steps:
+    - ${{ if and(eq(parameters.codesign, true), eq(parameters.publish_nuget, true)) }}:
+      - task: ManualValidation@1
+        inputs:
+          notifyUsers: ${{ variables.notifyUsers }}
+          approvers: ${{ variables.approvers }}
+    - ${{ if not(and(eq(parameters.codesign, true), eq(parameters.publish_nuget, true))) }}:
+      - script: echo "Skipping pre-deployment approval"
 
 # Build, sign dlls, build nuget pkgs, then sign them
 - job: Build
@@ -237,4 +240,4 @@ jobs:
           inputs:
             Contents: '**\*.dmp'
             TargetFolder: '$(Build.ArtifactStagingDirectory)/test_outputs_${{category}}_${{framework}}_$(Build.BuildId)'
-            OverWrite: true
+            OverWrite: true