Skip to content

Commit

Permalink
workflows/tests: fix template-injection zizmor info
Browse files Browse the repository at this point in the history
This updates `workflows/tests.yml` to use environment variables to
address `template-injection` info from `zizmor`.
  • Loading branch information
samford committed Dec 16, 2024
1 parent ce01bc7 commit a97fcdc
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -232,8 +232,12 @@ jobs:
bottles-directory: ${{ env.BOTTLES_DIR }}
cleanup: ${{ matrix.cleanup }}

- run: brew test-bot ${{ needs.setup_tests.outputs.test-bot-formulae-args }}
id: brew-test-bot-formulae
- id: brew-test-bot-formulae
run: |
# shellcheck disable=SC2086
brew test-bot $TEST_BOT_FORMULAE_ARGS
env:
TEST_BOT_FORMULAE_ARGS: ${{ needs.setup_tests.outputs.test-bot-formulae-args }}
working-directory: ${{ env.BOTTLES_DIR }}

- name: Post-build steps
Expand Down Expand Up @@ -355,7 +359,11 @@ jobs:
cleanup: ${{ matrix.cleanup }}
download-bottles: true

- run: brew test-bot ${{ needs.setup_dep_tests.outputs.test-bot-dependents-args }} --testing-formulae="$TESTING_FORMULAE"
- run: |
# shellcheck disable=SC2086
brew test-bot $TEST_BOT_DEPENDENTS_ARGS --testing-formulae="$TESTING_FORMULAE"
env:
TEST_BOT_DEPENDENTS_ARGS: ${{ needs.setup_dep_tests.outputs.test-bot-dependents-args }}
working-directory: ${{ env.BOTTLES_DIR }}
- name: Steps summary and cleanup
Expand Down

0 comments on commit a97fcdc

Please sign in to comment.