chore: add pre-commit hook and run linting in workflow (#228)

fastify · Oct 4, 2022 · 42b2356 · 42b2356
1 parent 067fc73
commit 42b2356
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -14,4 +14,5 @@ jobs:
   test:
     uses: fastify/workflows/.github/workflows/plugins-ci.yml@v3
     with:
+      lint: true
       license-check: true
diff --git a/package.json b/package.json
@@ -10,7 +10,7 @@
     "lint:fix": "standard --fix",
     "lint:standard": "standard",
     "lint:typescript": "standard --parser @typescript-eslint/parser --plugin @typescript-eslint/eslint-plugin test/*.ts",
-    "test": "npm run lint && npm run unit && npm run typescript",
+    "test": "npm run unit && npm run typescript",
     "test:ci": "npm run lint && tap test/*.test.js --coverage-report=lcovonly && npm run typescript",
     "typescript": "tsd",
     "unit": "tap test/*.test.js"
@@ -33,6 +33,7 @@
   },
   "homepage": "https://github.com/fastify/fastify-cors#readme",
   "devDependencies": {
+    "@fastify/pre-commit": "^2.0.2",
     "@types/node": "^18.0.0",
     "@typescript-eslint/eslint-plugin": "^5.12.1",
     "@typescript-eslint/parser": "^5.12.1",
@@ -52,5 +53,9 @@
   },
   "publishConfig": {
     "access": "public"
-  }
+  },
+  "pre-commit": [
+    "lint",
+    "test"
+  ]
 }