Skip to content

Commit afa6270

Browse files
committed
fix: modernize GHA workflow and set appropriate permissions
1 parent b508e5c commit afa6270

1 file changed

Lines changed: 18 additions & 11 deletions

File tree

‎.github/workflows/test.yml‎

Lines changed: 18 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,23 @@ name: Test
22

33
on: [push, pull_request]
44

5+
permissions:
6+
contents: read
7+
58
jobs:
69
tests:
710
runs-on: ubuntu-latest
811

912
strategy:
1013
matrix:
11-
node-version: [14.x, 15.x]
14+
node-version: [lts/*, latest]
1215

1316
steps:
14-
- uses: actions/checkout@v2
17+
- uses: actions/checkout@v7
18+
with:
19+
persist-credentials: false
1520
- name: Setup Node.js
16-
uses: actions/setup-node@v2.1.2
21+
uses: actions/setup-node@v7
1722
with:
1823
node-version: ${{ matrix.node-version }}
1924
- run: npm ci
@@ -23,23 +28,25 @@ jobs:
2328
# whenever changes are made that effects the `dist/index.js` bundle.
2429
release:
2530
needs: tests
31+
permissions:
32+
contents: write
2633
runs-on: ubuntu-latest
27-
if: github.ref == 'refs/heads/main'
34+
if: github.ref == 'refs/heads/main' && github.repository == 'nodejs/node-pr-labeler'
2835

2936
env:
3037
TAG: v1
3138

3239
steps:
33-
- uses: actions/checkout@v2
40+
- uses: actions/checkout@v7
41+
with:
42+
persist-credentials: true
3443
- name: Setup Node.js
35-
uses: actions/setup-node@v1
44+
uses: actions/setup-node@v7
3645
with:
37-
node-version: 14.x
46+
node-version: lts/*
3847

39-
- name: npm install and build
40-
run: |
41-
npm ci
42-
npm run build
48+
- run: npm ci
49+
- run: npm run build
4350

4451
- name: Check if dist/ has changed after build
4552
id: changedTest

0 commit comments

Comments
 (0)