Merge 8a7e097 into eda82af

nestjs · Mar 4, 2018 · 4b05325 · 4b05325
2 parents eda82af + 8a7e097
commit 4b05325
Show file tree

Hide file tree

Showing 53 changed files with 357 additions and 167 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -10,5 +10,6 @@ install:
      - npm install
      - gulp build
 script:
+      - npm run lint
       - npm test
 after_success: npm run coverage
diff --git a/examples/12-graphql-apollo/src/cats/cats.resolvers.ts b/examples/12-graphql-apollo/src/cats/cats.resolvers.ts
@@ -1,5 +1,11 @@
 import { Component, UseGuards } from '@nestjs/common';
-import { Query, Mutation, Resolver, DelegateProperty, Subscription } from '@nestjs/graphql';
+import {
+  Query,
+  Mutation,
+  Resolver,
+  DelegateProperty,
+  Subscription,
+} from '@nestjs/graphql';
 import { PubSub } from 'graphql-subscriptions';
 
 import { Cat } from './interfaces/cat.interface';

diff --git a/examples/12-graphql-apollo/src/cats/cats.service.ts b/examples/12-graphql-apollo/src/cats/cats.service.ts
@@ -5,7 +5,7 @@ import { Cat } from './interfaces/cat.interface';
 export class CatsService {
   private readonly cats: Cat[] = [{ id: 1, name: 'Cat', age: 5 }];
 
-  create(cat: Cat): Cat{
+  create(cat: Cat): Cat {
     this.cats.push(cat);
     return cat;
   }

diff --git a/examples/12-graphql-apollo/src/subscriptions/subscription.constants.ts b/examples/12-graphql-apollo/src/subscriptions/subscription.constants.ts
@@ -1 +1 @@
-export const SUBSCRIPTION_SERVER = 'SUBSCRIPTION_SERVER';
+export const SUBSCRIPTION_SERVER = 'SUBSCRIPTION_SERVER';
diff --git a/examples/12-graphql-apollo/src/subscriptions/subscription.providers.ts b/examples/12-graphql-apollo/src/subscriptions/subscription.providers.ts
@@ -7,9 +7,7 @@ export const createSubscriptionProviders = (port: number = 3001) => [
     provide: SUBSCRIPTION_SERVER,
     useFactory: () => {
       const server = createServer();
-      return new Promise(resolve =>
-        server.listen(port, () => resolve(server)),
-      );
+      return new Promise(resolve => server.listen(port, () => resolve(server)));
     },
   },
 ];
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -6,6 +6,7 @@
   "scripts": {
     "test": "nyc --require ts-node/register mocha src/**/*.spec.ts --reporter spec",
     "coverage": "nyc report --reporter=text-lcov | coveralls",
+    "lint": "tslint -p tsconfig.json -c tslint.json",
     "prettier": "prettier */**/*.ts --ignore-path ./.prettierignore --write && git status",
     "build": "gulp build && gulp move",
     "build:lib": "gulp build --dist lib",
@@ -78,7 +79,8 @@
     "prettier": "^1.9.2",
     "sinon": "^2.1.0",
     "sinon-chai": "^2.8.0",
-    "ts-node": "^3.2.0"
+    "ts-node": "^3.2.0",
+    "tslint": "^5.9.1"
   },
   "collective": {
     "type": "opencollective",

diff --git a/src/common/decorators/http/create-route-param-metadata.decorator.ts b/src/common/decorators/http/create-route-param-metadata.decorator.ts
@@ -13,7 +13,7 @@ const assignCustomMetadata = (
   index: number,
   factory: CustomParamFactory,
   data?: ParamData,
-  ...pipes: PipeTransform<any>[]
+  ...pipes: PipeTransform<any>[],
 ) => ({
   ...args,
   [`${paramtype}${CUSTOM_ROUTE_AGRS_METADATA}:${index}`]: {

diff --git a/src/common/decorators/http/index.ts b/src/common/decorators/http/index.ts
@@ -2,4 +2,4 @@ export * from './request-mapping.decorator';
 export * from './route-params.decorator';
 export * from './http-code.decorator';
 export * from './create-route-param-metadata.decorator';
-export * from './render.decorator';
+export * from './render.decorator';
diff --git a/src/common/decorators/http/route-params.decorator.ts b/src/common/decorators/http/route-params.decorator.ts
@@ -17,7 +17,7 @@ const assignMetadata = (
   paramtype: RouteParamtypes,
   index: number,
   data?: ParamData,
-  ...pipes: PipeTransform<any>[]
+  ...pipes: PipeTransform<any>[],
 ) => ({
   ...args,
   [`${paramtype}:${index}`]: {
@@ -41,7 +41,7 @@ const createRouteParamDecorator = (paramtype: RouteParamtypes) => {
 
 const createPipesRouteParamDecorator = (paramtype: RouteParamtypes) => (
   data?,
-  ...pipes: PipeTransform<any>[]
+  ...pipes: PipeTransform<any>[],
 ): ParameterDecorator => (target, key, index) => {
   const args = Reflect.getMetadata(ROUTE_ARGS_METADATA, target, key) || {};
   const hasParamData = isNil(data) || isString(data);
@@ -78,38 +78,35 @@ export const Headers: (
   property?: string,
 ) => ParameterDecorator = createRouteParamDecorator(RouteParamtypes.HEADERS);
 
-export function Query();
 export function Query(...pipes: PipeTransform<any>[]);
 export function Query(property: string, ...pipes: PipeTransform<any>[]);
 export function Query(
   property?: string | PipeTransform<any>,
-  ...pipes: PipeTransform<any>[]
+  ...pipes: PipeTransform<any>[],
 ) {
   return createPipesRouteParamDecorator(RouteParamtypes.QUERY)(
     property,
     ...pipes,
   );
 }
 
-export function Body();
 export function Body(...pipes: PipeTransform<any>[]);
 export function Body(property: string, ...pipes: PipeTransform<any>[]);
 export function Body(
   property?: string | PipeTransform<any>,
-  ...pipes: PipeTransform<any>[]
+  ...pipes: PipeTransform<any>[],
 ) {
   return createPipesRouteParamDecorator(RouteParamtypes.BODY)(
     property,
     ...pipes,
   );
 }
 
-export function Param();
 export function Param(...pipes: PipeTransform<any>[]);
 export function Param(property: string, ...pipes: PipeTransform<any>[]);
 export function Param(
   property?: string | PipeTransform<any>,
-  ...pipes: PipeTransform<any>[]
+  ...pipes: PipeTransform<any>[],
 ) {
   return createPipesRouteParamDecorator(RouteParamtypes.PARAM)(
     property,

diff --git a/src/common/enums/route-paramtypes.enum.ts b/src/common/enums/route-paramtypes.enum.ts
@@ -8,5 +8,5 @@ export enum RouteParamtypes {
   HEADERS,
   SESSION,
   FILE,
-  FILES
+  FILES,
 }
diff --git a/src/common/http/http.service.ts b/src/common/http/http.service.ts
@@ -7,48 +7,48 @@ import {
 import 'rxjs/add/observable/fromPromise';
 
 export class HttpService {
-  request<T = any>(config: AxiosRequestConfig): Observable<AxiosResponse<T>> {
+  public request<T = any>(config: AxiosRequestConfig): Observable<AxiosResponse<T>> {
     return Observable.fromPromise(axios.request<T>(config));
   }
-  
-  get<T = any>(
+
+  public get<T = any>(
     url: string,
     config?: AxiosRequestConfig,
   ): Observable<AxiosResponse<T>> {
     return Observable.fromPromise(axios.get<T>(url, config));
   }
 
-  delete(
+  public delete(
     url: string,
     config?: AxiosRequestConfig,
   ): Observable<AxiosResponse<any>> {
     return Observable.fromPromise(axios.delete(url, config));
   }
 
-  head(
+  public head(
     url: string,
     config?: AxiosRequestConfig,
   ): Observable<AxiosResponse<any>> {
     return Observable.fromPromise(axios.head(url, config));
   }
 
-  post(
+  public post(
     url: string,
     data?,
     config?: AxiosRequestConfig,
   ): Observable<AxiosResponse<any>> {
     return Observable.fromPromise(axios.post(url, data, config));
   }
 
-  put(
+  public put(
     url: string,
     data?,
     config?: AxiosRequestConfig,
   ): Observable<AxiosResponse<any>> {
     return Observable.fromPromise(axios.post(url, data, config));
   }
 
-  patch(
+  public patch(
     url: string,
     data?,
     config?: AxiosRequestConfig,

diff --git a/src/common/http/index.ts b/src/common/http/index.ts
@@ -1,2 +1,2 @@
 export * from './http.module';
-export * from './http.service';
+export * from './http.service';