Add DependsOn parameter to matrix job generation template#1527
Add DependsOn parameter to matrix job generation template#15271 commit merged intoAzure:masterfrom benbp:benbp/generator-dependson
Conversation
benbp
added
the
Central-EngSys
|
The following pipelines have been queued for testing: |
| type: object | ||
| - name: DependsOn | ||
| type: object | ||
| default: null |
There was a problem hiding this comment.
Is null a valid value? I don't see it mentioned anywhere at https://docs.microsoft.com/en-us/azure/devops/pipelines/process/templates?view=azure-devops. Should we just leave this as empty instead?
There was a problem hiding this comment.
The challenge I'm having is that leaving it empty marks it as a required parameter, but defaulting it to [] means my ${{ if ... }} conditional won't work. Perhaps there's a better way?
There was a problem hiding this comment.
Can't you just set it to '' and then compare against that string in your condition?
There was a problem hiding this comment.
But to your main question, it does work when empty. My understanding is it's a yaml object type, which I assume is a JSON object type, for which null is a valid field.
There was a problem hiding this comment.
As long as it works in your testing I'm fine with this approach I just haven't seen it before and couldn't find any reference to null in the docs.
weshaggard
left a comment
weshaggard
left a comment
There was a problem hiding this comment.
One question to investigate but otherwise looks good.
|
Hello @azure-sdk! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
ghost
merged commit 3 participants
This is a useful parameter for scenarios like CI pipelines where the test jobs make up a subset of jobs within a stage and need to depend on another job like
Build.