Skip to content

Commit

Permalink
Fix ensure Notify API key for each job
Browse files Browse the repository at this point in the history
I thought I fixed this in e7e86ee but
looks like I only did the Rails 7 job? I am working on the assumption
that once the runner has the env var set all jobs will work until the
runner shuts down as an unknown point - we'll have to see.
  • Loading branch information
mec committed Apr 5, 2024
1 parent 37e4ce3 commit 2780072
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/rails-integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,9 @@ jobs:
cache-to: type=gha,mode=min
-
name: Run integration tests
run: docker run --rm -e "NOTIFY_API_KEY=${{ secrets.NOTIFY_API_KEY }}" mail-notify-integration-rails-5:latest
env:
NOTIFY_API_KEY: ${{ secrets.NOTIFY_API_KEY }}
run: docker run --rm -e "NOTIFY_API_KEY=$NOTIFY_API_KEY" mail-notify-integration-rails-5:latest
rails-6:
name: Rails 6 mailer previews
runs-on: ubuntu-latest
Expand All @@ -60,7 +62,9 @@ jobs:
cache-to: type=gha,mode=min
-
name: Run integration tests
run: docker run --rm -e "NOTIFY_API_KEY=${{ secrets.NOTIFY_API_KEY }}" mail-notify-integration-rails-6:latest
env:
NOTIFY_API_KEY: ${{ secrets.NOTIFY_API_KEY }}
run: docker run --rm -e "NOTIFY_API_KEY=$NOTIFY_API_KEY" mail-notify-integration-rails-6:latest

rails-7:
name: Rails 7 mailer previews
Expand Down

0 comments on commit 2780072

Please sign in to comment.