Skip to content

Commit f77ec27

Browse files
azure-sdksima-zhu
andauthored
check the right variable syntax (Azure#21349)
Co-authored-by: sima-zhu <[email protected]>
1 parent 90ebed9 commit f77ec27

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,14 +30,14 @@ steps:
3030
displayName: CredScan setup
3131
- task: securedevelopmentteam.vss-secure-development-tools.build-task-credscan.CredScan@3
3232
displayName: CredScan running
33-
condition: and(succeededOrFailed(), ne('SKIP_CREDSCAN', true))
33+
condition: and(succeededOrFailed(), ne(variables['SKIP_CREDSCAN'], true))
3434
inputs:
3535
toolVersion: 2.2.7.8
3636
scanFolder: "${{ parameters.SourceDirectory }}/credscan.tsv"
3737
suppressionsFile: ${{ parameters.SuppressionFilePath }}
3838
- task: securedevelopmentteam.vss-secure-development-tools.build-task-postanalysis.PostAnalysis@2
3939
displayName: CredScan result analysis
40-
condition: and(succeededOrFailed(), ne('SKIP_CREDSCAN', true))
40+
condition: and(succeededOrFailed(), ne(variables['SKIP_CREDSCAN'], true))
4141
inputs:
4242
GdnBreakBaselineFiles: ${{ parameters.BaselineFilePath }}
4343
GdnBreakAllTools: false
@@ -50,4 +50,4 @@ steps:
5050
- pwsh: |
5151
Write-Host "Please check https://aka.ms/azsdk/credscan for more information about the cred scan failure."
5252
displayName: CredScan troubleshooting guide
53-
condition: and(failed(), ne('SKIP_CREDSCAN', true))
53+
condition: and(failed(), ne(variables['SKIP_CREDSCAN'], true))

0 commit comments

Comments
 (0)