Skip to content

Commit

Permalink
add the ability to override default succeeded() conditioning by param…
Browse files Browse the repository at this point in the history
…eter (#36421)

Co-authored-by: Scott Beddall <scbedd@microsoft.com>
  • Loading branch information
azure-sdk and scbedd authored Jul 10, 2024
1 parent 4079db4 commit 0e975bd
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
# is used when this pipeline is going to be generating and publishing daily dev builds.
parameters:
ServiceDirectory: ''
Condition: succeeded()
steps:
- ${{if ne(parameters.ServiceDirectory, '')}}:
- task: Powershell@2
Expand All @@ -13,12 +14,12 @@ steps:
pwsh: true
workingDirectory: $(Pipeline.Workspace)
displayName: Dump Package properties
condition: succeeded()
condition: ${{ parameters.Condition }}
- pwsh: |
$setDailyDevBuild = "false"
if (('$(Build.Reason)' -eq 'Schedule') -and ('$(System.TeamProject)' -eq 'internal')) {
$setDailyDevBuild = "true"
}
echo "##vso[task.setvariable variable=SetDevVersion]$setDailyDevBuild"
displayName: "Setup Versioning Properties"
condition: and(succeeded(), eq(variables['SetDevVersion'], ''))
condition: and(${{ parameters.Condition }}, eq(variables['SetDevVersion'], ''))
4 changes: 4 additions & 0 deletions eng/common/pipelines/templates/steps/verify-changelog.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,9 @@ parameters:
- name: ForRelease
type: boolean
default: false
- name: Condition
type: string
default: succeeded()

steps:
- task: Powershell@2
Expand All @@ -23,4 +26,5 @@ steps:
pwsh: true
workingDirectory: $(Pipeline.Workspace)
displayName: Verify ChangeLogEntry for ${{ parameters.PackageName }}
condition: ${{ parameters.Condition }}
continueOnError: false
4 changes: 3 additions & 1 deletion eng/common/pipelines/templates/steps/verify-path-length.yml
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
# Template for all Python Scripts in this repository
parameters:
parameters:
SourceDirectory: ''
BasePathLength: 49
Condition: succeeded()

steps:
- task: PythonScript@0
displayName: Analyze Path Lengths
condition: ${{ parameters.Condition }}
inputs:
scriptSource: inline
script: |
Expand Down
4 changes: 4 additions & 0 deletions eng/common/pipelines/templates/steps/verify-readme.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,14 @@ parameters:
- name: DocWardenVersion
type: string
default: ''
- name: Condition
type: string
default: succeeded()

steps:
- task: PowerShell@2
displayName: "Verify Readmes"
condition: ${{ parameters.Condition }}
inputs:
filePath: "eng/common/scripts/Verify-Readme.ps1"
arguments: >
Expand Down

0 comments on commit 0e975bd

Please sign in to comment.