ci: Resolve permission issue with release dry run and coverage report…

…ing (#1733) * ci: Resolve permission issue with release dry run and coverage reporting * key name * uses a dash
rigetti · Feb 6, 2024 · 5d6146d · 5d6146d
1 parent 4903d9d
commit 5d6146d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/release-dry-run.yml b/.github/workflows/release-dry-run.yml
@@ -17,10 +17,9 @@ jobs:
   release-dry-run:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
       with:
         fetch-depth: 0
-        token: ${{ secrets.PAT }}
     - name: Install Knope
       uses: knope-dev/[email protected]
       with:

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -113,6 +113,8 @@ jobs:
   test-unit:
     name: Test Unit
     runs-on: ubuntu-latest
+    permissions:
+      pull-requests: write # allows coverage bot to comment
     strategy:
       matrix:
         python-version: ["3.8", "3.9", "3.10", "3.11", "3.12"]
@@ -140,7 +142,7 @@ jobs:
           coverageFile: coverage.xml
           thresholdAll: 0.87
           thresholdNew: 0.9
-          token:  ${{ secrets.PAT }}
+          token:  ${{ secrets.GITHUB_TOKEN }}
 
   check-docker-image:
     name: Check types