ensure conditions all work properly in the case of a previous error (Azure#29423)

azure-sdk · scbedd · web-flow · commit c58e01ace40b · 2022-06-14T14:50:40.000-07:00
Co-authored-by: Scott Beddall &lt;scbedd@microsoft.com&gt;
diff --git a/eng/common/pipelines/templates/steps/credscan.yml b/eng/common/pipelines/templates/steps/credscan.yml
@@ -28,6 +28,8 @@ steps:
       Write-Host "##vso[task.setvariable variable=SKIP_CREDSCAN]true"
     }
   displayName: CredScan setup
+  condition: succeededOrFailed()
+
 - task: securedevelopmentteam.vss-secure-development-tools.build-task-credscan.CredScan@3
   displayName: CredScan running
   condition: and(succeededOrFailed(), ne(variables['SKIP_CREDSCAN'], true))

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,8 @@ steps:`
`28`	`28`	`Write-Host "##vso[task.setvariable variable=SKIP_CREDSCAN]true"`
`29`	`29`	`}`
`30`	`30`	`displayName: CredScan setup`
	`31`	`+ condition: succeededOrFailed()`
	`32`	`+`
`31`	`33`	`- task: securedevelopmentteam.vss-secure-development-tools.build-task-credscan.CredScan@3`
`32`	`34`	`displayName: CredScan running`
`33`	`35`	`condition: and(succeededOrFailed(), ne(variables['SKIP_CREDSCAN'], true))`