CI clean up to fix lint task and remove Windows / MacOS from matrix (brianc#3122)

* Remove unused travis CI config

* Bump eslint and friends

* Fix lint errors after eslint upgrade

* Remove windows and macos from CI workflow as they are actually running linux

Removes the windows and macos matrix from the CI workflow as they were never actually setting
the OS. Both were running against the "ubuntu-latest" OS. Trying to actually use them would
not work either as neither windows or macos is supported for service containers. A different
means will be needed to test on those platforms. Until that's done, this removes those from
the matrix as we were simply running the same thing 3x for the same node versions.

Author: sehrope

.github/workflows/ci.yml

@@ -21,7 +21,6 @@ jobs:
       - run: yarn lint
   build:
     needs: lint
-    runs-on: ubuntu-latest
     services:
       postgres:
         image: postgres:11
@@ -35,9 +34,16 @@ jobs:
         options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
     strategy:
       matrix:
-        node: ['10', '12', '14', '16', '18']
-        os: [ubuntu-latest, windows-latest, macos-latest]
+        node:
+          - '10'
+          - '12'
+          - '14'
+          - '16'
+          - '18'
+        os:
+          - ubuntu-latest
     name: Node.js ${{ matrix.node }} (${{ matrix.os }})
+    runs-on: ${{ matrix.os }}
     env:
         PGUSER: postgres
         PGHOST: localhost
@@ -47,6 +53,9 @@ jobs:
         SCRAM_TEST_PGUSER: scram_test
         SCRAM_TEST_PGPASSWORD: test4scram
     steps:
+      - name: Show OS
+        run: |
+          uname -a
       - run: |
           psql \
             -c "SET password_encryption = 'scram-sha-256'" \

.travis.yml

@@ -20,12 +20,12 @@
     "lint": "eslint '*/**/*.{js,ts,tsx}'"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^4.4.0",
-    "@typescript-eslint/parser": "^4.4.0",
-    "eslint": "^7.11.0",
+    "@typescript-eslint/eslint-plugin": "^6.17.0",
+    "@typescript-eslint/parser": "^6.17.0",
+    "eslint": "^8.56.0",
     "eslint-config-prettier": "^8.5.0",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-prettier": "^3.1.4",
+    "eslint-plugin-prettier": "^5.1.2",
     "lerna": "^3.19.0",
     "prettier": "3.0.3",
     "typescript": "^4.0.3"

package.json

@@ -111,14 +111,21 @@ export class DatabaseError extends Error implements NoticeOrError {
   public file: string | undefined
   public line: string | undefined
   public routine: string | undefined
-  constructor(message: string, public readonly length: number, public readonly name: MessageName) {
+  constructor(
+    message: string,
+    public readonly length: number,
+    public readonly name: MessageName
+  ) {
     super(message)
   }
 }
 
 export class CopyDataMessage {
   public readonly name = 'copyData'
-  constructor(public readonly length: number, public readonly chunk: Buffer) {}
+  constructor(
+    public readonly length: number,
+    public readonly chunk: Buffer
+  ) {}
 }
 
 export class CopyResponse {
@@ -148,15 +155,21 @@ export class Field {
 export class RowDescriptionMessage {
   public readonly name: MessageName = 'rowDescription'
   public readonly fields: Field[]
-  constructor(public readonly length: number, public readonly fieldCount: number) {
+  constructor(
+    public readonly length: number,
+    public readonly fieldCount: number
+  ) {
     this.fields = new Array(this.fieldCount)
   }
 }
 
 export class ParameterDescriptionMessage {
   public readonly name: MessageName = 'parameterDescription'
   public readonly dataTypeIDs: number[]
-  constructor(public readonly length: number, public readonly parameterCount: number) {
+  constructor(
+    public readonly length: number,
+    public readonly parameterCount: number
+  ) {
     this.dataTypeIDs = new Array(this.parameterCount)
   }
 }
@@ -172,12 +185,19 @@ export class ParameterStatusMessage {
 
 export class AuthenticationMD5Password implements BackendMessage {
   public readonly name: MessageName = 'authenticationMD5Password'
-  constructor(public readonly length: number, public readonly salt: Buffer) {}
+  constructor(
+    public readonly length: number,
+    public readonly salt: Buffer
+  ) {}
 }
 
 export class BackendKeyDataMessage {
   public readonly name: MessageName = 'backendKeyData'
-  constructor(public readonly length: number, public readonly processID: number, public readonly secretKey: number) {}
+  constructor(
+    public readonly length: number,
+    public readonly processID: number,
+    public readonly secretKey: number
+  ) {}
 }
 
 export class NotificationResponseMessage {
@@ -192,24 +212,36 @@ export class NotificationResponseMessage {
 
 export class ReadyForQueryMessage {
   public readonly name: MessageName = 'readyForQuery'
-  constructor(public readonly length: number, public readonly status: string) {}
+  constructor(
+    public readonly length: number,
+    public readonly status: string
+  ) {}
 }
 
 export class CommandCompleteMessage {
   public readonly name: MessageName = 'commandComplete'
-  constructor(public readonly length: number, public readonly text: string) {}
+  constructor(
+    public readonly length: number,
+    public readonly text: string
+  ) {}
 }
 
 export class DataRowMessage {
   public readonly fieldCount: number
   public readonly name: MessageName = 'dataRow'
-  constructor(public length: number, public fields: any[]) {
+  constructor(
+    public length: number,
+    public fields: any[]
+  ) {
     this.fieldCount = fields.length
   }
 }
 
 export class NoticeMessage implements BackendMessage, NoticeOrError {
-  constructor(public readonly length: number, public readonly message: string | undefined) {}
+  constructor(
+    public readonly length: number,
+    public readonly message: string | undefined
+  ) {}
   public readonly name = 'notice'
   public severity: string | undefined
   public code: string | undefined