From d4b34fc12fd6d3c127cc64e24686e165d62483d4 Mon Sep 17 00:00:00 2001 From: Michael Terry Date: Tue, 16 Jul 2024 08:53:49 -0400 Subject: [PATCH] ci: disable coverage checking action for now It has a permission issue with PRs from forks, and until we figure out how to fix that securely, let's just disable the check. --- .github/workflows/ci.yaml | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 3dddf5e11..560402fa6 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -41,14 +41,14 @@ jobs: run: | coverage report -m --skip-covered - - name: Check coverage report - # Disable coverage checking from forks until there is a secure fix for: - # https://github.com/orgoro/coverage/issues/259 - if: github.ref != 'refs/heads/main' && ${{ ! github.event.pull_request.head.repo.fork }} - uses: orgoro/coverage@v3.2 - with: - coverageFile: coverage.xml - token: ${{ secrets.GITHUB_TOKEN }} - thresholdAll: .45 - thresholdNew: 1 - thresholdModified: .40 +# Disable coverage checking until there is a secure fix for: +# https://github.com/orgoro/coverage/issues/259 +# - name: Check coverage report +# if: github.ref != 'refs/heads/main' +# uses: orgoro/coverage@v3.2 +# with: +# coverageFile: coverage.xml +# token: ${{ secrets.GITHUB_TOKEN }} +# thresholdAll: .45 +# thresholdNew: 1 +# thresholdModified: .40