Skip to content

Commit 17e90bc

Browse files
[release/dev17.14] [release/dev18.0] Fix parameter syntax in pipeline definition (#81297)
Backport of #81286 to release/dev17.14 /cc @JoeRobich @github-actions[bot] ## Customer Impact ## Regression - [ ] Yes - [ ] No [If yes, specify when the regression was introduced. Provide the PR or commit if known.] ## Testing [How was the fix verified? How was the issue missed previously? What tests were added?] ## Risk [High/Medium/Low. Justify the indication by mentioning how risks were measured and addressed.] --------- Co-authored-by: Jan Jones <janjones@microsoft.com>
1 parent 1ea49e8 commit 17e90bc

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

azure-pipelines-official.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -308,10 +308,10 @@ extends:
308308
-binaryLog
309309
-configuration $(BuildConfiguration)
310310
-officialBuildId $(Build.BuildNumber)
311-
-officialSkipTests $(SkipTests)
311+
-officialSkipTests ${{ parameters.SkipTests }}
312312
-officialSkipApplyOptimizationData ${{ parameters.SkipApplyOptimizationData }}
313313
-officialSourceBranchName $(SourceBranchName)
314-
-officialIbcDrop $(IbcDrop)
314+
-officialIbcDrop ${{ parameters.IbcDrop }}
315315
-officialVisualStudioDropAccessToken $(_DevDivDropAccessToken)
316316
/p:RepositoryName=$(Build.Repository.Name)
317317
/p:VisualStudioDropName=$(VisualStudio.DropName)
@@ -351,7 +351,7 @@ extends:
351351
testResultsFiles: '$(Build.SourcesDirectory)\artifacts\TestResults\$(BuildConfiguration)\*.xml'
352352
mergeTestResults: true
353353
testRunTitle: 'Unit Tests'
354-
condition: and(succeededOrFailed(), ne(variables['SkipTests'], 'true'))
354+
condition: and(succeededOrFailed(), ${{ not(parameters.SkipTests) }})
355355

356356
# Publish to Build Asset Registry
357357
- template: /eng/common/templates-official/job/publish-build-assets.yml@self

0 commit comments

Comments
 (0)