Skip to content

Commit 2a6d439

Browse files
Fix create-pull-request bug (Azure#12001)
Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
1 parent 122b9cd commit 2a6d439

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

eng/common/pipelines/templates/steps/create-pull-request.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,4 +86,4 @@ steps:
8686
-UserReviewers "${{ parameters.GHReviewersVariable }}"
8787
-TeamReviewers "${{ parameters.GHTeamReviewersVariable }}"
8888
-Assignees "${{ parameters.GHAssignessVariable }}"
89-
-CloseAfterOpenForTesting $${{ parameters.CloseAfterOpenForTesting }}
89+
-CloseAfterOpenForTesting $${{ coalesce(parameters.CloseAfterOpenForTesting, 'false') }}

eng/common/pipelines/templates/steps/docs-metadata-release.yml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ parameters:
1818
GHReviewersVariable: ''
1919
GHTeamReviewersVariable: '' # externally set, as eng-common does not have the identity-resolver. Run as pre-step
2020
OnboardingBranch: ''
21+
CloseAfterOpenForTesting: false
2122

2223
steps:
2324
- pwsh: |
@@ -83,7 +84,7 @@ steps:
8384
GHReviewersVariable: ${{ parameters.GHReviewersVariable }}
8485
GHTeamReviewersVariable: ${{ parameters.GHTeamReviewersVariable }}
8586
PRLabels: ${{ parameters.PRLabels }}
86-
CloseAfterOpenForTesting: $(TestPipeline)
87+
CloseAfterOpenForTesting: ${{ parameters.CloseAfterOpenForTesting }}
8788

8889
- ${{if ne( parameters['OnboardingBranch'], '')}}:
8990
- pwsh: |
@@ -124,4 +125,4 @@ steps:
124125
ScriptDirectory: ${{ parameters.WorkingDirectory }}/${{ parameters.ScriptDirectory }}
125126
GHReviewersVariable: ${{ parameters.GHReviewersVariable }}
126127
GHTeamReviewersVariable: ${{ parameters.GHTeamReviewersVariable }}
127-
CloseAfterOpenForTesting: $(TestPipeline)
128+
CloseAfterOpenForTesting: ${{ parameters.CloseAfterOpenForTesting }}

eng/common/scripts/Invoke-GitHubAPI.ps1

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ function Invoke-GitHubAPIPost {
3535
else {
3636
$warning = "{0} with Uri [ $apiURI ] did not fire request because of empty body." -f (Get-PSCallStack)[1].FunctionName
3737
LogWarning $warning
38-
exit 0
38+
return $null
3939
}
4040
}
4141
catch {
@@ -69,7 +69,7 @@ function Invoke-GitHubAPIPatch {
6969
else {
7070
$warning = "{0} with Uri [ $apiURI ] did not fire request because of empty body." -f (Get-PSCallStack)[1].FunctionName
7171
LogWarning $warning
72-
exit 0
72+
return $null
7373
}
7474
}
7575
catch {

0 commit comments

Comments
 (0)