From 2e6b2b4074842df846b0c2b0de97853f80d98e8f 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 | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/rails-integration-tests.yml b/.github/workflows/rails-integration-tests.yml index afe54a5..fbc2dda 100644 --- a/.github/workflows/rails-integration-tests.yml +++ b/.github/workflows/rails-integration-tests.yml @@ -41,7 +41,8 @@ jobs: file: ./Dockerfile build-args: | RAILS_VERSION=${{ matrix.rails }} - MAIL_NOTIFY_BRANCH=${{ github.ref }} + MAIL_NOTIFY_BRANCH=${{ github.head_ref || github.ref_name }} + RUBY_VERSION=3.2.6 push: false tags: mail-notify-integration-rails-${{ matrix.rails }}:latest outputs: type=docker, dest=/tmp/rails-${{ matrix.rails }}-image.tar