Skip to content

Commit c9f47f9

Browse files
azure-sdkweshaggardCopilot
authored
Sync eng/common directory with azure-sdk-tools for PR 13325 (#44401)
* Refactor API Review task conditions in YAML * Apply suggestion from @Copilot Co-authored-by: Copilot <[email protected]> --------- Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Copilot <[email protected]>
1 parent 541865b commit c9f47f9

File tree

1 file changed

+26
-28
lines changed

1 file changed

+26
-28
lines changed

eng/common/pipelines/templates/steps/create-apireview.yml

Lines changed: 26 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -37,31 +37,29 @@ steps:
3737
parameters:
3838
WorkingDirectory: ${{ parameters.SourceRootPath }}
3939

40-
- task: AzureCLI@2
41-
inputs:
42-
azureSubscription: 'APIView prod deployment'
43-
scriptType: pscore
44-
scriptLocation: scriptPath
45-
scriptPath: ${{ parameters.SourceRootPath }}/eng/common/scripts/Create-APIReview.ps1
46-
# PackageInfoFiles example: @('a/file1.json','a/file2.json')
47-
arguments: >
48-
-PackageInfoFiles @('${{ join(''',''', parameters.PackageInfoFiles) }}')
49-
-ArtifactList ('${{ convertToJson(parameters.Artifacts) }}' | ConvertFrom-Json | Select-Object Name)
50-
-ArtifactPath '${{parameters.ArtifactPath}}'
51-
-ArtifactName ${{ parameters.ArtifactName }}
52-
-PackageName '${{parameters.PackageName}}'
53-
-SourceBranch '$(Build.SourceBranchName)'
54-
-DefaultBranch '$(DefaultBranch)'
55-
-ConfigFileDir '${{parameters.ConfigFileDir}}'
56-
-BuildId '$(Build.BuildId)'
57-
-RepoName '$(Build.Repository.Name)'
58-
-MarkPackageAsShipped $${{parameters.MarkPackageAsShipped}}
59-
displayName: Create API Review
60-
condition: >-
61-
and(
62-
succeededOrFailed(),
63-
ne(variables['Skip.CreateApiReview'], 'true'),
64-
ne(variables['Build.Reason'],'PullRequest'),
65-
eq(variables['System.TeamProject'], 'internal'),
66-
not(endsWith(variables['Build.Repository.Name'], '-pr'))
67-
)
40+
- ${{ if and(eq(variables['System.TeamProject'], 'internal'), ne(variables['Build.Reason'], 'PullRequest'), not(endsWith(variables['Build.Repository.Name'], '-pr'))) }}:
41+
- task: AzureCLI@2
42+
inputs:
43+
azureSubscription: 'APIView prod deployment'
44+
scriptType: pscore
45+
scriptLocation: scriptPath
46+
scriptPath: ${{ parameters.SourceRootPath }}/eng/common/scripts/Create-APIReview.ps1
47+
# PackageInfoFiles example: @('a/file1.json','a/file2.json')
48+
arguments: >
49+
-PackageInfoFiles @('${{ join(''',''', parameters.PackageInfoFiles) }}')
50+
-ArtifactList ('${{ convertToJson(parameters.Artifacts) }}' | ConvertFrom-Json | Select-Object Name)
51+
-ArtifactPath '${{parameters.ArtifactPath}}'
52+
-ArtifactName ${{ parameters.ArtifactName }}
53+
-PackageName '${{parameters.PackageName}}'
54+
-SourceBranch '$(Build.SourceBranchName)'
55+
-DefaultBranch '$(DefaultBranch)'
56+
-ConfigFileDir '${{parameters.ConfigFileDir}}'
57+
-BuildId '$(Build.BuildId)'
58+
-RepoName '$(Build.Repository.Name)'
59+
-MarkPackageAsShipped $${{parameters.MarkPackageAsShipped}}
60+
displayName: Create API Review
61+
condition: >-
62+
and(
63+
succeededOrFailed(),
64+
ne(variables['Skip.CreateApiReview'], 'true')
65+
)

0 commit comments

Comments
 (0)