Skip to content

Commit

Permalink
Fix Rails integration Notify API key
Browse files Browse the repository at this point in the history
I am not too sure how this worked before, but we need to put the secret
into the runners environment so it can be passed by `docker run`.
  • Loading branch information
mec committed Apr 5, 2024
1 parent 35cd60b commit e7e86ee
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/rails-integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,4 +61,6 @@ 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-7:latest
env:
NOTIFY_API_KEY: ${{ secrets.NOTIFY_API_KEY }}
run: docker run --rm -e "NOTIFY_API_KEY=$NOTIFY_API_KEY" mail-notify-integration-rails-7:latest

0 comments on commit e7e86ee

Please sign in to comment.