From 1cd78c3dc2488502ff8e9af61cb067cbb4e83e7d Mon Sep 17 00:00:00 2001
From: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Date: Wed, 16 Mar 2022 11:55:13 -0700
Subject: [PATCH] Sync eng/common directory with azure-sdk-tools for PR 2901
 (#3437)

* skip credscanning when only file deletes are done
* applying condition wherever applicable
* updating name as accepted by the crew

Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com>
---
 eng/common/pipelines/templates/steps/credscan.yml | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/eng/common/pipelines/templates/steps/credscan.yml b/eng/common/pipelines/templates/steps/credscan.yml
index 202888e76..3f61883b6 100644
--- a/eng/common/pipelines/templates/steps/credscan.yml
+++ b/eng/common/pipelines/templates/steps/credscan.yml
@@ -25,15 +25,20 @@ steps:
     if(Test-Path "${{ parameters.SourceDirectory }}/credscan.tsv") {
       Get-Content "${{ parameters.SourceDirectory }}/credscan.tsv"
     }
+    else {
+      Write-Host "##vso[task.setvariable variable=SKIP_CREDSCAN]true"
+    }
   displayName: CredScan setup
 - task: securedevelopmentteam.vss-secure-development-tools.build-task-credscan.CredScan@3
   displayName: CredScan running
+  condition: and(succeededOrFailed(), ne('SKIP_CREDSCAN', true))
   inputs:
     toolVersion: 2.2.7.8 
     scanFolder: "${{ parameters.SourceDirectory }}/credscan.tsv"
     suppressionsFile: ${{ parameters.SuppressionFilePath }}
 - task: securedevelopmentteam.vss-secure-development-tools.build-task-postanalysis.PostAnalysis@2
   displayName: CredScan result analysis
+  condition: and(succeededOrFailed(), ne('SKIP_CREDSCAN', true))
   inputs:
     GdnBreakBaselineFiles: ${{ parameters.BaselineFilePath }}
     GdnBreakAllTools: false
@@ -43,8 +48,7 @@ steps:
     # Used for generating baseline file.
     # GdnBreakOutputBaselineFile: baseline
     # GdnBreakOutputBaseline: baseline
-  condition: succeededOrFailed()
 - pwsh: |
     Write-Host "Please check https://aka.ms/azsdk/credscan for more information about the cred scan failure."
   displayName: CredScan troubleshooting guide
-  condition: failed()
+  condition: and(failed(), ne('SKIP_CREDSCAN', true))