Skip to content

Commit b74432c

Browse files
committed
run lint & size as part of CI, but not before npm run test
The build step is no longer needed for tests, which in part allows us to drop these.
1 parent 21e83d3 commit b74432c

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

.github/workflows/nodejs.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,9 @@ jobs:
2626
node-version: 13.11.0
2727
- name: Install dependencies
2828
run: npm i
29+
- name: Lint Codebase
30+
run: npm run lint
2931
- name: Run Node.js Tests
3032
run: npm run test
33+
- name: Check Bundle Size
34+
run: npm run size

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
"postpublish": "npm publish --ignore-scripts --@github:registry='https://npm.pkg.github.com'",
3131
"presize": "npm run build",
3232
"size": "size-limit",
33-
"pretest": "npm run size && npm run lint",
3433
"test": "web-test-runner test/* --node-resolve"
3534
},
3635
"prettier": "@github/prettier-config",

0 commit comments

Comments
 (0)