Skip to content

Commit 2506070

Browse files
Apply review feedback: use npm ci and remove redundant festivals.json copy
Co-authored-by: richardthe3rd <573334+richardthe3rd@users.noreply.github.com>
1 parent 569cbe5 commit 2506070

2 files changed

Lines changed: 1 addition & 4 deletions

File tree

.github/workflows/deploy-worker.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,6 @@ jobs:
9191
working-directory: cloudflare-worker
9292
run: npm ci
9393

94-
- name: Copy festivals.json to worker directory
95-
run: cp data/festivals.json cloudflare-worker/festivals.json
96-
9794
- name: Run worker tests
9895
working-directory: cloudflare-worker
9996
run: npm test

mise.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,4 +48,4 @@ run = 'flutter analyze --no-fatal-infos'
4848
description = "Run Cloudflare Worker tests (Vitest + workerd)"
4949
dir = "cloudflare-worker"
5050
sources = ['cloudflare-worker/package.json', 'cloudflare-worker/package-lock.json', 'cloudflare-worker/worker.js', 'cloudflare-worker/test/**/*.js', 'cloudflare-worker/vitest.config.js', 'data/festivals.json']
51-
run = 'npm install && npm test'
51+
run = 'npm ci && npm test'

0 commit comments

Comments
 (0)