Skip to content

Commit cf032c1

Browse files
azuclaude
andcommitted
fix: add explicit permissions to test workflow
- Add `permissions: contents: read` to the test job to fix zizmor excessive-permissions warning - Pin action versions with commit SHA for security - Add persist-credentials: false to checkout action 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 3124338 commit cf032c1

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.github/workflows/test.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,18 @@ jobs:
44
test:
55
name: "Test on Node.js ${{ matrix.node-version }}"
66
runs-on: ubuntu-latest
7+
permissions:
8+
contents: read
79
strategy:
810
matrix:
911
node-version: [ 18, 20 ]
1012
steps:
1113
- name: checkout
12-
uses: actions/checkout@v3
14+
uses: actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744 # v3.6.0
15+
with:
16+
persist-credentials: false
1317
- name: setup Node.js ${{ matrix.node-version }}
14-
uses: actions/setup-node@v3
18+
uses: actions/setup-node@3235b876344d2a9aa001b8d1453c930bba69e610 # v3.9.1
1519
with:
1620
node-version: ${{ matrix.node-version }}
1721
- name: Install

0 commit comments

Comments
 (0)