Skip to content

Commit

Permalink
Merge pull request #1727 from terrestris/gh-action-escape
Browse files Browse the repository at this point in the history
Escape github action conditions
  • Loading branch information
LukasLohoff authored Aug 6, 2024
2 parents 63b4bb2 + d5f8474 commit d9dba45
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/on-pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
npx jest --config=jest.config.js --coverage --coverageReporters json-summary --changedSince=main --coverageDirectory ./coverage/changed
- name: Check for test results ❓
if: github.actor != 'dependabot[bot]'
if: ${{ github.actor != 'dependabot[bot]' }}
id: check-results
run: |
if [ -f ./coverage/changed/coverage-summary.json ]; then
Expand All @@ -60,7 +60,7 @@ jobs:
fi
- name: Construct jest coverage comment input 💬
if: github.actor != 'dependabot[bot]'
if: ${{ github.actor != 'dependabot[bot]' }}
id: construct-input
run: |
if [ "${{ env.no_tests_found }}" == "true" ]; then
Expand All @@ -70,7 +70,7 @@ jobs:
fi
- name: Jest Coverage Comment 💬
if: github.actor != 'dependabot[bot]'
if: ${{ github.actor != 'dependabot[bot]' }}
uses: MishaKav/jest-coverage-comment@main
with:
hide-comment: false
Expand All @@ -79,7 +79,7 @@ jobs:
multiple-files: ${{ env.MULTIPLE_FILES }}

- name: Get Coverage Comment Id ➡️
if: env.no_tests_found == 'true' && github.actor != 'dependabot[bot]'
if: ${{ env.no_tests_found == 'true' && github.actor != 'dependabot[bot]' }}
id: get-comment-id
run: |
PR_NUMBER=${{ github.event.pull_request.number }}
Expand All @@ -91,7 +91,7 @@ jobs:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- name: Post To No Tests Found Comment ⚠️
if: env.no_tests_found == 'true' && github.actor != 'dependabot[bot]'
if: ${{ env.no_tests_found == 'true' && github.actor != 'dependabot[bot]' }}
uses: peter-evans/create-or-update-comment@v4
with:
comment-id: ${{ env.COVERAGE_COMMENT_ID }}
Expand Down

0 comments on commit d9dba45

Please sign in to comment.