Skip to content

Commit

Permalink
Fix trigger all tests branch (#4135)
Browse files Browse the repository at this point in the history
Fix trigger all tests branch (again). 

This was a pretty obvious bug - I was using the output of the
`get-branch` step, but I never set the id of the step to that.

This setup now more closely matches the [solution laid out as a github
comment](CircleCI-Public/trigger-circleci-pipeline-action#61 (comment))
  • Loading branch information
aboedo authored Jul 30, 2024
1 parent 51fabaa commit 781151d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/trigger_all_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
# As of writing, this is installed in ubuntu-latest
# https://github.com/actions/runner-images/blob/main/images/ubuntu/Ubuntu2204-Readme.md#cli-tools

- name: 'Get PR branch'
- id: 'get-branch' # the id here is important since it gets reused in the next step
run: echo ::set-output name=branch::$(gh pr view $PR_NO --repo $REPO --json headRefName --jq '.headRefName')
env:
REPO: ${{ github.repository }}
Expand Down

0 comments on commit 781151d

Please sign in to comment.