Skip to content

Commit 1f97f2f

Browse files
Don't run Roslyn analyzers on PRs (#336)
* Don't run Roslyn analyzers on PRs * fix
1 parent 5a43822 commit 1f97f2f

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

builds/azure-pipelines/template-steps-build-test.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,12 +92,13 @@ steps:
9292
arguments: 'analyze $(Build.SourcesDirectory)\src\bin\${{ parameters.configuration }}\* --recurse --verbose'
9393
condition: and(succeeded(), eq(variables['Agent.OS'], 'Windows_NT'))
9494

95+
# Don't run for PRs since this currently breaks on runs from forks. We run this daily ourselves anyways.
9596
- task: securedevelopmentteam.vss-secure-development-tools.build-task-roslynanalyzers.RoslynAnalyzers@3
9697
inputs:
9798
userProvideBuildInfo: 'autoMsBuildInfo'
9899
env:
99100
SYSTEM_ACCESSTOKEN: $(System.AccessToken)
100-
condition: and(succeeded(), eq(variables['Agent.OS'], 'Windows_NT'))
101+
condition: and(succeeded(), eq(variables['Agent.OS'], 'Windows_NT'), ne(variables['Build.Reason'], 'PullRequest'))
101102

102103
- task: securedevelopmentteam.vss-secure-development-tools.build-task-credscan.CredScan@2
103104
inputs:

0 commit comments

Comments
 (0)