Skip to content

Commit

Permalink
fix: add explicit permissions to CI.yml (#755)
Browse files Browse the repository at this point in the history
* chore: add explicit permissions to CI.yml now that we have moved to enterprise

* using permissions from semantic-release example

* remove contents:write as it is thought to be for github packages
  • Loading branch information
sookburt committed May 26, 2023
1 parent b4edc70 commit f692a4d
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,9 @@ jobs:
needs: [build, test-e2e, test-unit, lint, types]
if: github.ref == 'refs/heads/main'
runs-on: ubuntu-latest
permissions:
pull-requests: write # to be able to comment on released pull requests
id-token: write # to enable use of OIDC for npm provenance
steps:
- name: Checkout
uses: actions/checkout@v3
Expand Down

2 comments on commit f692a4d

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.❔
Category Percentage Covered / Total
🟒 Statements 92.12% 222/241
🟒 Branches 85.44% 88/103
🟒 Functions 88.71% 55/62
🟒 Lines 91.81% 213/232

Test suite run success

327 tests passing in 15 suites.

Report generated by πŸ§ͺjest coverage report action from f692a4d

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.❔
Category Percentage Covered / Total
🟒 Statements 92.12% 222/241
🟒 Branches 85.44% 88/103
🟒 Functions 88.71% 55/62
🟒 Lines 91.81% 213/232

Test suite run success

327 tests passing in 15 suites.

Report generated by πŸ§ͺjest coverage report action from f692a4d

Please sign in to comment.