Remove failing unicorn tests

Also clarify module testing by removing lockfile.

Author: bcomnes

.github/workflows/external.yml

@@ -77,7 +77,7 @@ jobs:
             ${{ runner.os }}-node-
 
       - name: install main
-        run: npm ci --no-audit --no-fund
+        run: npm i --no-audit --no-fund
         working-directory: ./main
       - name: install project
         run: '[ -f package-lock.json ] && npm ci --force || npm install --force'

.github/workflows/npm-test.yml

@@ -28,6 +28,5 @@ jobs:
       uses: actions/setup-node@v3
       with:
         node-version: ${{ matrix.node_version }}
-        cache: 'npm'
-    - run: npm ci
+    - run: npm i
     - run: npm test

.gitignore

@@ -2,5 +2,6 @@
 /node_modules
 /yarn.lock
 /.vscode
+package-lock.json
 
 # Library specific ones

index.js

@@ -14,8 +14,7 @@ module.exports = {
   },
   'plugins': [
     '@typescript-eslint',
-    'react-hooks',
-    'unicorn'
+    'react-hooks'
   ],
   'extends': ['./lib/standard-eslint-ts'],
   'settings': {
@@ -65,8 +64,6 @@ module.exports = {
     'react-hooks/rules-of-hooks': 'off',
     'react-hooks/exhaustive-deps': 'off',
 
-    'unicorn/expiring-todo-comments': 'warn',
-
     '@typescript-eslint/consistent-type-imports': [
       'error',
       { 'prefer': 'type-imports', 'disallowTypeAnnotations': true }