Tweak Workflows to trigger coverage creation and ensure permissions a…

…re set
jetstack · Aug 27, 2024 · 81c83ba · 81c83ba
1 parent 45fd679
commit 81c83ba
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/.github/workflows/build-test.yaml b/.github/workflows/build-test.yaml
@@ -3,6 +3,8 @@ on:
   pull_request:
     branches:
       - 'main'
+    paths:
+      - "!README.md"
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -58,10 +60,13 @@ jobs:
     - name: Run all unit tests
       run: make test
 
-    - name: Check test coverage
-      run: |
-        go install github.com/vladopajic/go-test-coverage/v2@latest
-        go-test-coverage --config=./.testcoverage.yml
+    - name: check test coverage
+      uses: vladopajic/go-test-coverage@v2
+      with:
+        config: ./.testcoverage.yml
+
+    - name: Trigger Coverage update
+      uses: ./coverage-badge.yaml
 
     - name: Generate code coverage artifacts
       uses: actions/upload-artifact@v4

diff --git a/.github/workflows/coverage-badge.yaml b/.github/workflows/coverage-badge.yaml
@@ -2,9 +2,10 @@ name: Generate code coverage badge
 
 on:
   workflow_dispatch: # Here for Testing
-  push:
-    branches:
-      - main
+  workflow_call:
+
+permissions:
+  contents: write
 
 jobs:
   test: