From 781151df49f570b070c02961ccdfc1147707be5d Mon Sep 17 00:00:00 2001 From: Andy Boedo Date: Tue, 30 Jul 2024 13:37:30 -0300 Subject: [PATCH] Fix trigger all tests branch (#4135) 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](https://github.com/CircleCI-Public/trigger-circleci-pipeline-action/issues/61#issuecomment-1662021882) --- .github/workflows/trigger_all_tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/trigger_all_tests.yml b/.github/workflows/trigger_all_tests.yml index d02439e43e..ce19ef0a77 100644 --- a/.github/workflows/trigger_all_tests.yml +++ b/.github/workflows/trigger_all_tests.yml @@ -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 }}