From 95f57cdf58f07b437d1149fdce8b871a22d58401 Mon Sep 17 00:00:00 2001 From: meyric Date: Fri, 13 Dec 2024 14:58:26 +0000 Subject: [PATCH] Change how we specify the gem branch We install the gem to run our integration tests. They recently started to fail and the issues seemed to be that bundler couldn't locate the branch. The branch name is supplied by `github.ref` switching to the logic here seems to have fixed this. `head_ref` for PRs with a fall back to `ref_name` if it is not a PR. Documentation: https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/accessing-contextual-information-about-workflow-runs#github-context --- .github/workflows/rails-integration-tests.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/rails-integration-tests.yml b/.github/workflows/rails-integration-tests.yml index afe54a5..8f7d2ad 100644 --- a/.github/workflows/rails-integration-tests.yml +++ b/.github/workflows/rails-integration-tests.yml @@ -41,7 +41,7 @@ jobs: file: ./Dockerfile build-args: | RAILS_VERSION=${{ matrix.rails }} - MAIL_NOTIFY_BRANCH=${{ github.ref }} + MAIL_NOTIFY_BRANCH=${{ github.head_ref || github.ref_name }} push: false tags: mail-notify-integration-rails-${{ matrix.rails }}:latest outputs: type=docker, dest=/tmp/rails-${{ matrix.rails }}-image.tar