Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/nestjs/nest
Browse files Browse the repository at this point in the history
  • Loading branch information
kamilmysliwiec committed Oct 6, 2018
2 parents b2f3dfe + 656ecf8 commit 1eef513
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions .travis.yml
Expand Up @@ -19,5 +19,6 @@ install:
- npm ci
- gulp build
script:
- npm run lint
- npm test
after_success: npm run coverage
4 changes: 2 additions & 2 deletions packages/core/injector/module-token-factory.ts
Expand Up @@ -2,7 +2,7 @@ import { DynamicModule } from '@nestjs/common';
import { SHARED_MODULE_METADATA } from '@nestjs/common/constants';
import { Type } from '@nestjs/common/interfaces/type.interface';
import * as hash from 'object-hash';
import safeStringify from 'fast-safe-stringify';
import stringify from 'fast-safe-stringify';

export class ModuleTokenFactory {
public create(
Expand All @@ -25,7 +25,7 @@ export class ModuleTokenFactory {
): string {
// Uses safeStringify instead of JSON.stringify
// to support circular dynamic modules
return dynamicModuleMetadata ? safeStringify(dynamicModuleMetadata) : '';
return dynamicModuleMetadata ? stringify(dynamicModuleMetadata) : '';
}

public getModuleName(metatype: Type<any>): string {
Expand Down

0 comments on commit 1eef513

Please sign in to comment.