Skip to content

Commit

Permalink
ci: add pull-request/write permission to app-test-coverage.yml and di…
Browse files Browse the repository at this point in the history
…sabled step with TODO
  • Loading branch information
DominikPinsel committed Oct 6, 2023
1 parent 82ddbd7 commit 34400ff
Showing 1 changed file with 13 additions and 10 deletions.
23 changes: 13 additions & 10 deletions .github/workflows/app-test-coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ on:
jobs:
test-app:
runs-on: ubuntu-latest
permissions:
pull-requests: write
steps:
- name: Checkout
uses: actions/checkout@v3
Expand Down Expand Up @@ -77,13 +79,14 @@ jobs:
if: success() || failure()
run: cat coveragereport/SummaryGithub.md >> $GITHUB_STEP_SUMMARY

- name: Publish code coverage report as PR comment
uses: madrapps/[email protected]
if: github.event_name == 'pull_request' && (success() || failure())
with:
paths: |
${{ github.workspace }}/build/reports/xml/jacoco
token: ${{ secrets.GITHUB_TOKEN }}
min-coverage-overall: 80
min-coverage-changed-files: 80
title: Coverage Report
# TODO Ask Committers to allow writes from PRs before enabling again
# - name: Publish code coverage report as PR comment
# uses: madrapps/[email protected]
# if: github.event_name == 'pull_request' && (success() || failure())
# with:
# paths: |
# ${{ github.workspace }}/build/reports/xml/jacoco
# token: ${{ secrets.GITHUB_TOKEN }}
# min-coverage-overall: 80
# min-coverage-changed-files: 80
# title: Coverage Report

0 comments on commit 34400ff

Please sign in to comment.