Skip to content

Commit e61c807

Browse files
committed
Add linters
1 parent 0920706 commit e61c807

2 files changed

Lines changed: 18 additions & 9 deletions

File tree

.github/workflows/test.yml

Lines changed: 16 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,9 @@ permissions:
3838
statuses: 'write'
3939

4040
jobs:
41+
lint:
42+
uses: 'google-github-actions/.github/.github/workflows/lint.yml@sethvargo/lint' # ratchet:exclude
43+
4144
unit:
4245
name: 'unit'
4346
runs-on: 'ubuntu-latest'
@@ -47,7 +50,7 @@ jobs:
4750

4851
- uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
4952
with:
50-
node-version: '20.x'
53+
node-version-file: 'package.json'
5154

5255
- name: 'npm build'
5356
run: 'npm ci && npm run build'
@@ -63,7 +66,8 @@ jobs:
6366
# Direct Workload Identity Federation
6467
#
6568
direct_workload_identity_federation:
66-
if: ${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
69+
if: |-
70+
${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
6771
name: 'direct_workload_identity_federation'
6872
runs-on: '${{ matrix.os }}'
6973
strategy:
@@ -82,7 +86,7 @@ jobs:
8286

8387
- uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
8488
with:
85-
node-version: '20.x'
89+
node-version-file: 'package.json'
8690

8791
- name: 'npm build'
8892
run: 'npm ci && npm run build'
@@ -116,7 +120,8 @@ jobs:
116120
# Workload Identity Federation through a Service Account
117121
#
118122
workload_identity_federation_through_service_account:
119-
if: ${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
123+
if: |-
124+
${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
120125
name: 'workload_identity_federation_through_service_account'
121126
runs-on: '${{ matrix.os }}'
122127
strategy:
@@ -135,7 +140,7 @@ jobs:
135140

136141
- uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
137142
with:
138-
node-version: '20.x'
143+
node-version-file: 'package.json'
139144

140145
- name: 'npm build'
141146
run: 'npm ci && npm run build'
@@ -187,7 +192,8 @@ jobs:
187192
# Service Account Key JSON
188193
#
189194
credentials_json:
190-
if: ${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
195+
if: |-
196+
${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
191197
name: 'credentials_json'
192198
runs-on: '${{ matrix.os }}'
193199
strategy:
@@ -203,7 +209,7 @@ jobs:
203209

204210
- uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
205211
with:
206-
node-version: '20.x'
212+
node-version-file: 'package.json'
207213

208214
- name: 'npm build'
209215
run: 'npm ci && npm run build'
@@ -254,7 +260,8 @@ jobs:
254260
# has permissions to read the file.
255261
#
256262
docker:
257-
if: ${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
263+
if: |-
264+
${{ github.event_name == 'push' || github.repository == github.event.pull_request.head.repo.full_name }}
258265
name: 'docker'
259266
runs-on: 'ubuntu-latest'
260267
strategy:
@@ -264,7 +271,7 @@ jobs:
264271

265272
- uses: 'actions/setup-node@1d0ff469b7ec7b3cb9d8673fde0c81c44821de2a' # ratchet:actions/setup-node@v4
266273
with:
267-
node-version: '20.x'
274+
node-version-file: 'package.json'
268275

269276
- name: 'npm build'
270277
run: 'npm ci && npm run build'

bin/runTests.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,6 @@ set -eEuo pipefail
1616
FILES="$(node -e "process.stdout.write(require('node:fs').readdirSync('./', { recursive: true }).filter((e) => {return e.endsWith('.test.ts') && !e.startsWith('node_modules');}).sort().join(' '));")"
1717

1818
set -x
19+
20+
# shellcheck disable=SC2086
1921
exec node --require ts-node/register --test-reporter spec --test ${FILES}

0 commit comments

Comments
 (0)