chore: add prettier

Author: DanielSchaffer

.eslintrc

@@ -1,9 +1,7 @@
 {
-  "extends": [
-    "plugin:@typescript-eslint/recommended"
-  ],
+  "extends": ["plugin:@typescript-eslint/recommended"],
   "parser": "@typescript-eslint/parser",
-  "plugins": ["import", "@typescript-eslint", "mocha"],
+  "plugins": ["import", "@typescript-eslint", "mocha", "prettier"],
   "parserOptions": {
     "ecmaVersion": 8
   },
@@ -49,7 +47,7 @@
     {
       "files": ["test/**/*.js"],
       "rules": {
-         "@typescript-eslint/no-var-requires": "off"
+        "@typescript-eslint/no-var-requires": "off"
       }
     }
   ],
@@ -132,32 +130,31 @@
     "space-in-parens": ["error", "never"],
     "spaced-comment": ["error", "always"],
 
-    "import/order": ["error", {
-      "alphabetize": {
-        "order": "asc",
-        "caseInsensitive": false
-      },
-      "groups": [
-        "builtin",
-        "external",
-        "internal",
-        "parent",
-        "sibling",
-        "index"
-      ],
-      "newlines-between":  "always-and-inside-groups"
-    }],
+    "import/order": [
+      "error",
+      {
+        "alphabetize": {
+          "order": "asc",
+          "caseInsensitive": false
+        },
+        "groups": ["builtin", "external", "internal", "parent", "sibling", "index"],
+        "newlines-between": "always-and-inside-groups"
+      }
+    ],
 
     "@typescript-eslint/ban-types": ["off"],
     "@typescript-eslint/class-name-casing": ["error"],
-    "@typescript-eslint/explicit-function-return-type": ["error", { "allowExpressions":  true }],
+    "@typescript-eslint/explicit-function-return-type": ["error", { "allowExpressions": true }],
     "@typescript-eslint/generic-type-naming": ["error", "^T(?:[A-Z][a-zA-Z]+)*$"],
     "@typescript-eslint/indent": ["off"],
-    "@typescript-eslint/member-delimiter-style": ["error", { "multiline": { "delimiter":  "none"} }],
+    "@typescript-eslint/member-delimiter-style": ["error", { "multiline": { "delimiter": "none" } }],
     "@typescript-eslint/no-inferrable-types": ["error", { "ignoreParameters": true, "ignoreProperties": true }],
     "@typescript-eslint/no-explicit-any": ["off"],
     "@typescript-eslint/no-unused-vars": ["error"],
-    "@typescript-eslint/no-use-before-define": ["error", { "functions": false, "classes": false, "variables": true, "enums": true }],
+    "@typescript-eslint/no-use-before-define": [
+      "error",
+      { "functions": false, "classes": false, "variables": true, "enums": true }
+    ],
     "@typescript-eslint/semi": ["error", "never"]
   }
 }

.mocharc.js

@@ -1,62 +1,40 @@
-// extension:
-//   - ts
-// require:
-//   - 'tsconfig-paths/register'
-//   - './test/ts-node.js'
-//   - 'ts-custom-error-shim'
-//   - './test/mocha.config'
-//   - 'test/sandbox.ts'
-// ignore:
-//   - node_modules
-// spec:
-//   - 'packages/*/*/{,(!node_modules)/}index.ts'
-//   - 'packages/*/*/{,(!node_modules)/*/}*/index.ts'
-//   - 'packages/*/*/{,(!node_modules)/}src/**/*.spec.ts'
-//   - 'packages/*/*/{,(!node_modules)/*/}*/src/**/*.spec.ts'
-//   - 'packages/*/*/{,(!node_modules)/}src/**/*.int-spec.ts'
-//   - 'packages/*/*/{,(!node_modules)/*/}*/src/**/*.int-spec.ts'
-
 const { readdirSync, statSync } = require('fs')
 const { dirname, resolve } = require('path')
 
-const [,, scopesArg, ...scopes] = process.argv
+const [, , scopesArg, ...scopes] = process.argv
 if (scopesArg !== '--scope') {
   scopes.length = 0
 }
 
-const manifest = require('./.tsconfig.builder.json').include.filter(file => file.endsWith('index.ts'))
+const manifest = require('./.tsconfig.builder.json').include.filter((file) => file.endsWith('index.ts'))
 function findPaths(dir) {
   const subDirs = readdirSync(dir)
   return subDirs
-    .filter(subDir => subDir !== 'src' && subDir !== 'node_modules' && subDir !== '.yarn-cache')
-    .map(subDir => `${dir}/${subDir}`)
-    .filter(subDir => statSync(subDir).isDirectory())
+    .filter((subDir) => subDir !== 'src' && subDir !== 'node_modules' && subDir !== '.yarn-cache')
+    .map((subDir) => `${dir}/${subDir}`)
+    .filter((subDir) => statSync(subDir).isDirectory())
     .reduce((result, subDir) => {
       result.push(subDir)
       result.push(...findPaths(subDir))
       return result
     }, [])
 }
 const paths = manifest
-  .filter(path => {
+  .filter((path) => {
     if (!scopes.length) {
       return true
     }
-    return scopes.some(scope => path.startsWith(`packages/${scope}/`))
+    return scopes.some((scope) => path.startsWith(`packages/${scope}/`))
   })
   .reduce((result, file) => {
     const dir = dirname(file)
     result.push(dir, ...findPaths(resolve(__dirname, dir)))
     return result
   }, [])
-const barrels = paths.map(path => `${path}/index.ts`)
-const unitSpec = paths.map(path => `${path}/src/**/*.spec.ts`)
-const intSpec = paths.map(path => `${path}/src/**/*.int-spec.ts`)
-const spec = [
-  ...barrels,
-  ...unitSpec,
-  ...intSpec,
-]
+const barrels = paths.map((path) => `${path}/index.ts`)
+const unitSpec = paths.map((path) => `${path}/src/**/*.spec.ts`)
+const intSpec = paths.map((path) => `${path}/src/**/*.int-spec.ts`)
+const spec = [...barrels, ...unitSpec, ...intSpec]
 
 // suppress "Cannot find any files matching pattern" warnings from mocha startup
 console.__ogWarn = console.warn
@@ -69,25 +47,14 @@ console.warn = (msg, ...args) => {
 
 module.exports = {
   extension: 'ts',
-  ignore: [
-    '.out',
-    '.build',
-    'dist',
-  ],
-  watchIgnore: [
-    '.out',
-    '.build',
-    'dist',
-  ],
+  ignore: ['.out', '.build', 'dist'],
+  watchIgnore: ['.out', '.build', 'dist'],
   require: [
     'tsconfig-paths/register',
     resolve(__dirname, './test/ts-node.js'),
     'ts-custom-error-shim',
     resolve(__dirname, './test/mocha.config'),
   ],
-  file: [
-    resolve(__dirname, 'test/sandbox.ts'),
-    ...manifest,
-  ],
+  file: [resolve(__dirname, 'test/sandbox.ts'), ...manifest],
   spec,
 }

.prettierignore

@@ -0,0 +1,10 @@
+.coverage
+.modules
+.nyc_output
+.prettierignore
+.yarn-cache
+*.pug
+*.tf
+node_modules
+out
+yarn.lock

.prettierrc

@@ -0,0 +1,6 @@
+{
+  "printWidth": 120,
+  "semi": false,
+  "singleQuote": true,
+  "trailingComma": "all"
+}

.travis.yml

@@ -16,13 +16,13 @@ cache:
     - packages/dandi-contrib/mvc-view-pug/node_modules
 
 install:
-  - "yarn setup"
-  - "yarn tsc --version"
-  - "yarn ts-node --version"
+  - 'yarn setup'
+  - 'yarn tsc --version'
+  - 'yarn ts-node --version'
 
 script:
-  - "yarn build"
-  - "yarn coverage"
+  - 'yarn build'
+  - 'yarn coverage'
 
 after_script:
-  - "yarn coverage-report"
+  - 'yarn coverage-report'

README.md

@@ -13,6 +13,7 @@ the features they require.
 [Get Started](./packages/dandi/core)
 
 ## Use Cases
+
 - MVC-style Application Servers
 - CLI Applications
 - AWS Lambda functions
@@ -32,7 +33,7 @@ the features they require.
 - Automatic output content negotiation based on request `accept` header
 - Create and configure interdependent middleware handlers (`HttpPipelinePreparer`) and they will automatically be
   invoked in the correct order
-- Add global transformers to control response data structure  
+- Add global transformers to control response data structure
 
 ### MVC
 
@@ -104,7 +105,8 @@ implementation of a very simple REST API using `@dandi`
 ## Dev Setup
 
 To set up this project as a local repository:
+
 - [Install Yarn Classic](https://classic.yarnpkg.com/en/docs/install) if you don't have it installed
 - After cloning the repostory, run `yarn setup` - this will install the dependencies for all the individual packages,
-  as well as the Dandi builder 
-- You can now run `yarn build` to build all packages, or `yarn test` to run all tests
+  as well as the Dandi builder
+- You can now run `yarn build` to build all packages, or `yarn test` to run all tests

_config.yml

@@ -1 +1 @@
-theme: jekyll-theme-minimal
+theme: jekyll-theme-minimal

_examples/simple-aws-lambda/serverless.yml

@@ -65,9 +65,9 @@ plugins:
 functions:
   helloWorld:
     handler: main.helloWorldHandler
-#    The following are a few example events you can configure
-#    NOTE: Please make sure to change your handler code to work with those events
-#    Check the event documentation for details
+    #    The following are a few example events you can configure
+    #    NOTE: Please make sure to change your handler code to work with those events
+    #    Check the event documentation for details
     events:
       - http:
           path: hello-world

_examples/simple-aws-lambda/src/data-processor.service.ts

@@ -6,9 +6,8 @@ const FAKE_LATENCY = 250
 
 @Injectable()
 export class DataProcessorService {
-
   public processData(data: ReceiveDataModel): Promise<ReceiveDataResponse> {
-    return new Promise<ReceiveDataResponse>(resolve => {
+    return new Promise<ReceiveDataResponse>((resolve) => {
       setTimeout(() => resolve(this.convertData(data)), FAKE_LATENCY)
     })
   }
@@ -19,5 +18,4 @@ export class DataProcessorService {
       modelType: data.constructor.name,
     }
   }
-
 }

_examples/simple-aws-lambda/src/hello-world.handler.ts

@@ -4,12 +4,10 @@ import { QueryParam } from '@dandi/http-model'
 
 @Injectable()
 export class HelloWorldHandler implements LambdaHandler {
-
   public handleEvent(@QueryParam(Number) foo: string): any {
     return {
       message: 'hello!',
       foo,
     }
   }
-
 }

_examples/simple-aws-lambda/src/receive-data.handler.spec.ts

@@ -10,19 +10,14 @@ import { ReceiveDataHandler } from './receive-data.handler'
 import { ReceiveDataResponse } from './receive-data.model'
 
 describe('ReceiveDataHandler', () => {
-
   let classUnderTest: ReceiveDataHandler
   let body: any
   let request: HttpRequest
 
-  const harness = testHarness(ReceiveDataHandler,
-    DataProcessorService,
-    ModelBuilderModule,
-    {
-      provide: HttpRequest,
-      useFactory: () => request,
-    },
-  )
+  const harness = testHarness(ReceiveDataHandler, DataProcessorService, ModelBuilderModule, {
+    provide: HttpRequest,
+    useFactory: () => request,
+  })
 
   beforeEach(async () => {
     body = {
@@ -45,7 +40,6 @@ describe('ReceiveDataHandler', () => {
 
   describe('handleEvent', () => {
     it('converts the incoming model to the response model', async () => {
-
       const result: ReceiveDataResponse = await harness.invoke(classUnderTest, 'handleEvent')
 
       expect(result).to.haveOwnProperty('model')
@@ -55,8 +49,6 @@ describe('ReceiveDataHandler', () => {
         message: 'hi!',
       })
       expect(result.modelType).to.equal('ReceiveDataModel')
-
     })
   })
-
 })

_examples/simple-aws-lambda/src/receive-data.handler.ts

@@ -7,11 +7,9 @@ import { ReceiveDataModel, ReceiveDataResponse } from './receive-data.model'
 
 @Injectable()
 export class ReceiveDataHandler implements LambdaHandler {
-
   constructor(@Inject(DataProcessorService) private processor: DataProcessorService) {}
 
   public handleEvent(@RequestBody(ReceiveDataModel) data: ReceiveDataModel): Promise<ReceiveDataResponse> {
     return this.processor.processData(data)
   }
-
 }

_examples/simple-console/cli.ts

@@ -7,10 +7,6 @@ import * as program from 'commander'
 import { run } from './src/main'
 import { PACKAGE_VERSION } from './src/package'
 
-program
-  .version(PACKAGE_VERSION)
-  .usage('[options]')
-  .parse(process.argv)
+program.version(PACKAGE_VERSION).usage('[options]').parse(process.argv)
 
 run(start, program)
-

_examples/simple-console/src/app-options.ts

@@ -3,21 +3,18 @@ import { InjectionToken, Provider } from '@dandi/core'
 import { localOpinionatedToken } from './local-token'
 
 // eslint-disable-next-line @typescript-eslint/no-empty-interface
-export interface AppOptions {
-}
+export interface AppOptions {}
 
 // eslint-disable-next-line @typescript-eslint/no-empty-interface
-export interface AppConfig {
-}
+export interface AppConfig {}
 
 export const AppConfig: InjectionToken<AppConfig> = localOpinionatedToken('AppConfig', {
   multi: false,
 })
 
 // eslint-disable-next-line @typescript-eslint/no-unused-vars
 export function installerConfigFactory(options: AppConfig): AppConfig {
-  return {
-  }
+  return {}
 }
 
 export function appConfigProvider(options: AppOptions): Provider<AppConfig> {

_examples/simple-console/src/app.ts

@@ -5,7 +5,6 @@ const SIM_FAKE_WORK_TIMEOUT = 50
 
 @Injectable(EntryPoint)
 export class SimpleConsoleApp implements EntryPoint {
-
   constructor(@Inject(Logger) private logger: Logger) {}
 
   public async run(): Promise<void> {
@@ -26,7 +25,6 @@ export class SimpleConsoleApp implements EntryPoint {
   }
 
   private fakeWork(timeout: number): Promise<void> {
-    return new Promise(resolve => setTimeout(resolve, timeout))
+    return new Promise((resolve) => setTimeout(resolve, timeout))
   }
-
 }