More TS migration steps

Author: Valentin Vasilyev

.DS_Store

@@ -25,4 +25,6 @@ jobs:
       with:
         node-version: ${{ matrix.node-version }}
     - run: yarn
+    - run: yarn lint
+    - run: yarn build
     - run: yarn test

.github/workflows/node.js.yml

@@ -1,4 +1,4 @@
-Nice PG SQL  toolkit
+Nice PG SQL toolkit
 ============================
 
 🧰 Tiny SQL toolkit for PG + Node (<200 LOC)

README.md

@@ -1,3 +1,3 @@
 module.exports = {
-  testEnvironment: "node",
-};
+  testEnvironment: 'node',
+}

jest.config.js

@@ -1,7 +1,7 @@
 {
   "name": "nice-pg-sql-toolkit",
   "version": "0.5.0",
-  "description": "Nice PG SQL toolkit. Loves SQL. Not an ORM. Can do migrations",
+  "description": "Nice PG SQL toolkit. Loves SQL. Not an ORM.",
   "main": "dist/index.ts",
   "types": "dist/index.d.ts",
   "repository": "https://github.com/fingerprintjs/nice-pg-sql-toolkit",

package.json

@@ -170,7 +170,7 @@ export const mapFromColumns = (row: Record<string, unknown>, map: Record<string,
   }, {})
 }
 
-class UniqueIndexError extends Error {
+export class UniqueIndexError extends Error {
   readonly table: string
   readonly constraint: string
   readonly columns: string[]

src/index.ts

@@ -1,21 +1,21 @@
-const {UniqueIndexError} = require("..//src/index");
+const { UniqueIndexError } = require('../dist/index')
 
-test("it parses single column constraint", () => {
+test('it parses single column constraint', () => {
   let attrs = {
-    table: "test_table",
-    constraint: "test_constraint",
-    detail: "Key (email)=([email protected]) already exists."
-  };
+    table: 'test_table',
+    constraint: 'test_constraint',
+    detail: 'Key (email)=([email protected]) already exists.',
+  }
   let err = new UniqueIndexError(attrs)
-  expect(err.columns).toEqual(["email"]);
-});
+  expect(err.columns).toEqual(['email'])
+})
 
-test("it parses multiple column constraint", () => {
+test('it parses multiple column constraint', () => {
   let attrs = {
-    table: "test_table",
-    constraint: "test_constraint",
-    detail: "Key (customer_id, display_name)=(customer1234, Corporation1) already exists."
-  };
+    table: 'test_table',
+    constraint: 'test_constraint',
+    detail: 'Key (customer_id, display_name)=(customer1234, Corporation1) already exists.',
+  }
   let err = new UniqueIndexError(attrs)
-  expect(err.columns).toEqual(["customer_id", "display_name"]);
-});
+  expect(err.columns).toEqual(['customer_id', 'display_name'])
+})

tests/index.test.js

@@ -7,7 +7,9 @@
     "declaration": true,
     "outDir": "./dist",
     "noUnusedLocals": true,
-    "noUnusedParameters": true
+    "noUnusedParameters": true,
+    "noImplicitAny": true,
+    "noImplicitReturns": true
   },
   "include": ["./src/**/*"],
   "exclude": ["./node_modules", "./tests/**/*"]