chore: Add linting

Author: kevin-greene-ck

package.json

@@ -10,17 +10,22 @@
   "scripts": {
     "clean": "rimraf dist/",
     "clean-all": "rimraf dist/ node_modules/ package-lock.json",
+    "lint": "tslint --fix './src/**/*.ts'",
     "docker": "docker-compose up --force-recreate &",
     "docker:kill": "docker-compose kill",
-    "prebuild": "npm run clean",
+    "prebuild": "npm run clean && npm run lint",
     "build": "tsc",
     "precopy-test-config": "rimraf dist/tests/integration/config",
     "copy-test-config": "cp -r src/tests/integration/config dist/tests/integration/config",
     "pretest": "npm run docker:kill && npm run build && npm run copy-test-config",
     "test": "npm run test:integration",
     "pretest:integration": "npm run docker",
     "test:integration": "wait-on --timeout 20000 http://localhost:8510 && node dist/tests/integration/bootstrap.js && CONSUL_ADDRESS='http://localhost:8510' CONSUL_DC='dc1' CONFIG_PATH='dist/tests/integration/config' lab --timeout 15000 --verbose -l -S -P spec dist/tests/integration",
-    "posttest:integration": "npm run docker:kill"
+    "posttest:integration": "npm run docker:kill",
+    "release:patch": "npm version patch && npm run release:publish",
+    "release:minor": "npm version minor && npm run release:publish",
+    "release:major": "npm version major && npm run release:publish",
+    "release:publish": "git push --follow-tags"
   },
   "author": "Credit Karma",
   "license": "Apache-2.0",

src/main/index.ts

@@ -1,2 +1,2 @@
 export * from './toggles'
-export * from './types'
+export * from './types'

src/main/schema.ts

@@ -7,23 +7,23 @@ export const toggleSchema: object = {
                 type: 'object',
                 properties: {
                     id: {
-                        type: 'string'
+                        type: 'string',
                     },
                     description: {
-                        type: 'string'
+                        type: 'string',
                     },
                     fraction: {
                         type: 'number',
                         minimum: 0.0,
-                        maximum: 1.0
+                        maximum: 1.0,
                     },
                     comment: {
-                        type: 'string'
-                    }
+                        type: 'string',
+                    },
                 },
-                required: [ 'id', 'fraction' ]
-            }
-        }
+                required: [ 'id', 'fraction' ],
+            },
+        },
     },
-    required: [ 'toggles' ]
-}
+    required: [ 'toggles' ],
+}

src/main/toggles.ts

@@ -1,13 +1,13 @@
 import { config } from '@creditkarma/dynamic-config'
-import * as logger from './logger'
 import { DEFAULT_TOGGLES_PATH } from './constants'
-import { objectMatchesSchema, memoize } from './utils'
+import * as logger from './logger'
 import { toggleSchema } from './schema'
 import {
-    ToggleMap,
     IToggleDescription,
     Toggle,
+    ToggleMap,
 } from './types'
+import { memoize, objectMatchesSchema } from './utils'
 
 const rawToggles: ToggleMap = new Map()
 
@@ -43,4 +43,4 @@ export function toggleMap(key: string): Promise<Toggle> {
             }
         }
     })
-}
+}

src/main/types.ts

@@ -7,4 +7,4 @@ export interface IToggleDescription {
     comment?: string
 }
 
-export type Toggle = () => boolean
+export type Toggle = () => boolean

src/main/utils.ts

@@ -7,7 +7,7 @@ export function objectMatchesSchema(schema: object, data: any): boolean {
 }
 
 export function memoize<T>(fn: () => T): () => T {
-    let cachedValue: any = undefined
+    let cachedValue: any
 
     return (): T => {
         if (cachedValue !== undefined) {
@@ -18,4 +18,4 @@ export function memoize<T>(fn: () => T): () => T {
             return cachedValue
         }
     }
-}
+}

src/tests/integration/bootstrap.ts

@@ -9,20 +9,20 @@ setTimeout(() => {
     Promise.all([
         consulClient.set({ path: 'toggles' }, [
             {
-                "id": "com.creditkarma.featureFlags.AlwaysEnabled",
-                "description": "It's a feature toggle",
-                "fraction": 1.0
+                id: 'com.creditkarma.featureFlags.AlwaysEnabled',
+                description: "It's a feature toggle",
+                fraction: 1.0,
             },
             {
-                "id": "com.creditkarma.featureFlags.AlwaysDisabled",
-                "description": "It's a feature toggle",
-                "fraction": 0
+                id: 'com.creditkarma.featureFlags.AlwaysDisabled',
+                description: "It's a feature toggle",
+                fraction: 0,
             },
             {
-                "id": "com.creditkarma.featureFlags.SometimesDisabled",
-                "description": "It's a feature toggle",
-                "fraction": 0.4
-            }
+                id: 'com.creditkarma.featureFlags.SometimesDisabled',
+                description: "It's a feature toggle",
+                fraction: 0.4,
+            },
         ]),
     ]).then(
         (result: any) => {

src/tests/integration/index.spec.ts

@@ -18,4 +18,4 @@ describe('ToggleMap', () => {
         const toggle = await toggleMap('com.creditkarma.featureFlags.AlwaysEnabled')
         expect(toggle()).to.equal(true)
     })
-})
+})