Skip to content

Commit c8d3e25

Browse files
azure-sdkscbedd
andauthored
Sync eng/common directory with azure-sdk-tools for PR 2901 (Azure#20881)
* skip credscanning when only file deletes are done * applying condition wherever applicable * updating name as accepted by the crew Co-authored-by: scbedd <[email protected]>
1 parent e274129 commit c8d3e25

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

eng/common/pipelines/templates/steps/credscan.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,15 +25,20 @@ steps:
2525
if(Test-Path "${{ parameters.SourceDirectory }}/credscan.tsv") {
2626
Get-Content "${{ parameters.SourceDirectory }}/credscan.tsv"
2727
}
28+
else {
29+
Write-Host "##vso[task.setvariable variable=SKIP_CREDSCAN]true"
30+
}
2831
displayName: CredScan setup
2932
- task: securedevelopmentteam.vss-secure-development-tools.build-task-credscan.CredScan@3
3033
displayName: CredScan running
34+
condition: and(succeededOrFailed(), ne('SKIP_CREDSCAN', true))
3135
inputs:
3236
toolVersion: 2.2.7.8
3337
scanFolder: "${{ parameters.SourceDirectory }}/credscan.tsv"
3438
suppressionsFile: ${{ parameters.SuppressionFilePath }}
3539
- task: securedevelopmentteam.vss-secure-development-tools.build-task-postanalysis.PostAnalysis@2
3640
displayName: CredScan result analysis
41+
condition: and(succeededOrFailed(), ne('SKIP_CREDSCAN', true))
3742
inputs:
3843
GdnBreakBaselineFiles: ${{ parameters.BaselineFilePath }}
3944
GdnBreakAllTools: false
@@ -43,8 +48,7 @@ steps:
4348
# Used for generating baseline file.
4449
# GdnBreakOutputBaselineFile: baseline
4550
# GdnBreakOutputBaseline: baseline
46-
condition: succeededOrFailed()
4751
- pwsh: |
4852
Write-Host "Please check https://aka.ms/azsdk/credscan for more information about the cred scan failure."
4953
displayName: CredScan troubleshooting guide
50-
condition: failed()
54+
condition: and(failed(), ne('SKIP_CREDSCAN', true))

0 commit comments

Comments
 (0)