Skip to content

Commit e6df3b3

Browse files
TerrenceMcGuinness-NOAATerry McGuinness
and
Terry McGuinness
authored
CI GitHub pipeline (hotfix) update for fetching repo name (NOAA-EMC#3084)
# Description<!-- This description will become the commit message for the PR. --> Simple bug fix to GitHub CI pipeline script for Parallel Works. Introduced a bug on its last update for setting the repo name in advance of running the GitHub CLI command for getting the repo owner and branch name of PRs. # Type of change - [x] Bug fix (fixes something broken) - [ ] New feature (adds functionality) - [ ] Maintenance (code refactor, clean-up, new CI test, etc.) # Change characteristics <!-- Choose YES or NO from each of the following and delete the other --> - Is this a breaking change (a change in existing functionality)? NO - Does this change require a documentation update? NO - Does this change require an update to any of the following submodules? # How has this been tested? Gets tested when update to default repo gets checked in. --------- Co-authored-by: Terry McGuinness <Terry.McGuinness@noaa.gov>
1 parent 3f68397 commit e6df3b3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/pw_aws_ci.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,9 @@ jobs:
7272
id: get-branch
7373
run: |
7474
pr_number=${{ github.event.inputs.pr_number }}
75+
repo=${{ github.repository }}
7576
if [ "$pr_number" -eq "0" ]; then
7677
branch=${{ github.event.inputs.ref }}
77-
repo=${{ github.repository }}
7878
else
7979
branch=$(gh pr view $pr_number --repo $repo --json headRefName --jq '.headRefName')
8080
repo_owner=$(gh pr view $pr_number --repo $repo --json headRepositoryOwner --jq '.headRepositoryOwner.login')

0 commit comments

Comments
 (0)