Skip to content

Commit eb33b96

Browse files
authored
Add explicit component governance build step (#22486)
- #22243 - ignore a number of artifacts/ folders that cause task warnings but don't add value - separately, correct overuse of `$(BuildDirectory)` variable
1 parent cd81d48 commit eb33b96

File tree

1 file changed

+14
-2
lines changed

1 file changed

+14
-2
lines changed

.azure/pipelines/jobs/default-build.yml

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ jobs:
164164
displayName: Install JDK 11
165165
- ${{ if eq(parameters.isTestingJob, true) }}:
166166
- powershell: |
167-
Write-Host "##vso[task.setvariable variable=SeleniumProcessTrackingFolder]$(BuildDirectory)\artifacts\tmp\selenium\"
167+
Write-Host "##vso[task.setvariable variable=SeleniumProcessTrackingFolder]$(Build.SourcesDirectory)\artifacts\tmp\selenium\"
168168
./eng/scripts/InstallGoogleChrome.ps1
169169
displayName: Install Chrome
170170
- ${{ if and(eq(parameters.installTar, 'true'), eq(parameters.agentOs, 'Windows')) }}:
@@ -238,13 +238,25 @@ jobs:
238238
continueOnError: true
239239
condition: always()
240240

241+
- ${{ if and(startsWith(parameters.jobDisplayName, 'Build:'), ne(variables['skipComponentGovernanceDetection'], 'true'), notin(variables['Build.Reason'], 'PullRequest')) }}:
242+
- task: ms.vss-governance-buildtask.governance-build-task-component-detection.ComponentGovernanceComponentDetection@0
243+
condition: and(succeeded(), ne(variables['CG_RAN'], 'true'))
244+
displayName: 'Component Detection'
245+
inputs:
246+
${{ if eq(parameters.agentOs, 'Windows') }}:
247+
ignoreDirectories: '.dotnet,.packages,artifacts\log,artifacts\symbols,artifacts\tmp'
248+
${{ if ne(parameters.agentOs, 'Windows') }}:
249+
ignoreDirectories: '.dotnet,.packages,artifacts/log,artifacts/symbols,artifacts/tmp'
250+
sourceScanPath: $(Build.SourcesDirectory)
251+
verbosity: Verbose
252+
241253
- ${{ each artifact in parameters.artifacts }}:
242254
- task: PublishBuildArtifacts@1
243255
displayName: Upload artifacts from ${{ artifact.path }}
244256
condition: and(or(succeeded(), eq('${{ artifact.publishOnError }}', 'true')), or(eq(variables['system.pullrequest.isfork'], false), eq('${{ artifact.includeForks }}', 'true')))
245257
continueOnError: true
246258
inputs:
247-
pathtoPublish: $(BuildDirectory)/${{ artifact.path }}
259+
pathtoPublish: $(Build.SourcesDirectory)/${{ artifact.path }}
248260
${{ if eq(artifact.name, '') }}:
249261
artifactName: artifacts-$(AgentOsName)-$(BuildConfiguration)
250262
${{ if ne(artifact.name, '') }}:

0 commit comments

Comments
 (0)