Clean up

- Separate lint/build/test methods
- Add linting step to CI

Author: cblanc

.github/workflows/ci.yml

@@ -32,7 +32,10 @@ jobs:
         run: npm run build
 
       - name: Test
-        run: npm run test
+        run: npm test
+
+      - name: Lint
+        run: npm run lint
 
       - name: Coverage
         run: npm run coverage

.github/workflows/release.yml

@@ -11,7 +11,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x]
+        node-version: [14.x]
 
     steps:
       - uses: actions/checkout@v1
@@ -33,6 +33,9 @@ jobs:
       - name: Test
         run: npm test
 
+      - name: Lint
+        run: npm run lint
+
       - name: Build
         run: npm run build
 

package.json

@@ -22,12 +22,12 @@
     "semantic-release": "semantic-release --no-ci",
     "test:watch": "NODE_ENV=test mocha --watch",
     "test:nock:reload": "NODE_ENV=test RECORD=true mocha",
-    "test": "npm run lint && NODE_ENV=test nyc mocha",
+    "test": "NODE_ENV=test nyc mocha",
     "lint": "eslint lib/**/*.ts",
-    "build": "npm run lint && tsc",
+    "build": "tsc",
     "coverage": "nyc report --reporter=text-lcov > coverage.lcov && codecov",
     "docs": "typedoc --mode file --out docs/ --name 'Ideal Postcodes Node.js Client' --theme minimal --excludePrivate --hideGenerator lib/",
-    "prepublishOnly": "npm run test && npm run build"
+    "prepublishOnly": "npm run build"
   },
   "files": [
     "dist"