Skip to content

Commit

Permalink
CI GitHub pipeline (hotfix) update for fetching repo name (#3084)
Browse files Browse the repository at this point in the history
# 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 <[email protected]>
  • Loading branch information
1 parent 3f68397 commit e6df3b3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/pw_aws_ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -72,9 +72,9 @@ jobs:
id: get-branch
run: |
pr_number=${{ github.event.inputs.pr_number }}
repo=${{ github.repository }}
if [ "$pr_number" -eq "0" ]; then
branch=${{ github.event.inputs.ref }}
repo=${{ github.repository }}
else
branch=$(gh pr view $pr_number --repo $repo --json headRefName --jq '.headRefName')
repo_owner=$(gh pr view $pr_number --repo $repo --json headRepositoryOwner --jq '.headRepositoryOwner.login')
Expand Down

0 comments on commit e6df3b3

Please sign in to comment.