Merge pull request #3 from hapinessjs/next

release(version): v1.0.0

Author: akanass

.nvmrc

@@ -1 +1 @@
-8.2.1
+9.3.0

Makefile

@@ -1,7 +1,7 @@
 pretest:
 	@node ./node_modules/.bin/tslint -p ./tsconfig.json --type-check "./src/**/*.ts" "./test/**/*.ts"
 test:
-	@node ./node_modules/.bin/istanbul cover ./node_modules/.bin/_mocha ./test
+	@node ./node_modules/.bin/istanbul cover ./node_modules/.bin/_mocha "./test/**/*.ts"
 coveralls:
 	cat ./coverage/lcov.info | node ./node_modules/.bin/coveralls
 tsc:

README.md

@@ -59,8 +59,8 @@ $ yarn add @hapiness/core @hapiness/date
 
 ```javascript
 "dependencies": {
-    "@hapiness/core": "^1.0.0-rc.6",
-    "@hapiness/date": "^1.0.0-rc.6",
+    "@hapiness/core": "^1.3.0",
+    "@hapiness/date": "^1.0.0",
     //...
 }
 //...
@@ -107,19 +107,18 @@ To set up your development environment:
 
 ## Change History
 
-* v1.0.0-rc.6 (2017-07-31)
+* v1.0.0 (2017-12-19)
     * Module initialization.
     * Moment.js helper + timezone.
     * Documentation.
-    * Module version related to core version.
 
 [Back to top](#table-of-contents)
 
 ## Maintainers
 
 <table>
     <tr>
-        <td colspan="4" align="center"><a href="https://www.tadaweb.com"><img src="https://tadaweb.com/images/tadaweb/logo.png" width="117" alt="tadaweb" /></a></td>
+        <td colspan="4" align="center"><a href="https://www.tadaweb.com"><img src="http://bit.ly/2xHQkTi" width="117" alt="tadaweb" /></a></td>
     </tr>
     <tr>
         <td align="center"><a href="https://github.com/Juneil"><img src="https://avatars3.githubusercontent.com/u/6546204?v=3&s=117" width="117"/></a></td>

package.json

@@ -1,8 +1,8 @@
 {
     "name": "@hapiness/date",
-    "version": "1.0.0-rc.6",
+    "version": "1.0.0",
     "description": "Date module for Hapiness framework based on moment",
-    "main": "index.js",
+    "main": "commonjs/index.js",
     "types": "index.d.ts",
     "private": false,
     "scripts": {
@@ -67,31 +67,29 @@
     },
     "homepage": "https://github.com/hapinessjs/date-module#readme",
     "peerdependencies": {
-        "@hapiness/core": "^1.0.0-rc.6"
+        "@hapiness/core": "^1.3.0"
     },
     "devDependencies": {
-        "@hapiness/core": "^1.0.0-rc.6",
-        "@types/fs-extra": "^4.0.0",
-        "coveralls": "^2.13.1",
-        "fs-extra": "^4.0.0",
-        "intercept-stdout": "^0.1.2",
+        "@hapiness/core": "^1.3.0",
+        "@types/fs-extra": "^5.0.0",
+        "coveralls": "^3.0.0",
+        "fs-extra": "^5.0.0",
         "istanbul": "^1.1.0-alpha.1",
-        "mocha": "^3.4.2",
-        "mocha-typescript": "^1.1.7",
-        "rimraf": "^2.6.1",
-        "rxjs": "^5.4.2",
+        "mocha": "^4.0.1",
+        "mocha-typescript": "^1.1.12",
+        "rimraf": "^2.6.2",
+        "rxjs": "^5.5.5",
         "ts-node": "^3.3.0",
-        "tslint": "^5.5.0",
-        "typescript": "^2.4.2",
+        "tslint": "^5.8.0",
+        "typescript": "^2.6.2",
         "unit.js": "^2.0.0"
     },
     "engines": {
         "node": ">=7.0.0"
     },
     "dependencies": {
-        "@types/moment": "^2.13.0",
-        "@types/moment-timezone": "^0.2.34",
-        "moment": "^2.18.1",
-        "moment-timezone": "^0.5.13"
+        "@types/moment-timezone": "^0.5.2",
+        "moment": "^2.20.1",
+        "moment-timezone": "^0.5.14"
     }
 }

src/module/date.module.ts

@@ -2,7 +2,7 @@ import { HapinessModule } from '@hapiness/core';
 import { DateService } from './date.service';
 
 @HapinessModule({
-    version: '1.0.0-rc.6',
+    version: '1.0.0',
     exports: [ DateService ]
 })
 export class DateModule {}

src/module/date.service.ts

@@ -1,4 +1,4 @@
-import { Optional, Inject, Injectable } from '@hapiness/core';
+import { Injectable } from '@hapiness/core';
 import * as momentTz from 'moment-timezone';
 import { Moment } from 'moment';
 import * as moment from 'moment';
@@ -17,7 +17,7 @@ export class DateService {
     /**
      * Get date helper
      */
-    helper = moment
+    helper = moment;
 
     /**
      * Get timezone infos

test/mocha.opts

@@ -2,6 +2,6 @@
 --report lcovonly
 --reporter spec
 --recursive
---compilers ts:ts-node/register
+--require ts-node/register
 --throw-deprecation
 --colors

tools/files.json

@@ -1,10 +1,5 @@
 [
     { "name":"README.md" },
     { "name":"LICENSE.md" },
-    { "name":"package.json" },
-    { "name":"src" },
-    { "name":"index.js", "remove":true },
-    { "name":"index.js.map", "remove":true },
-    { "name":"index.d.ts", "remove":true },
-    { "name":"module", "remove":true }
+    { "name":"package.json" }
 ]

tools/packaging.ts

@@ -9,8 +9,6 @@ import * as fs from 'fs-extra';
  */
 interface FileObject {
     name: string;
-    remove?: boolean;
-    externals?: boolean;
 }
 
 /**
@@ -41,28 +39,23 @@ class Packaging {
      * Function to copy one file
      *
      * @param file {string}
-     * @param externals {boolean}
      *
-     * @return {Observable<R>}
+     * @return {Observable<any>}
      */
-    private _copy(file: string, externals?: boolean): Observable<any> {
+    private _copy(file: string): Observable<any> {
         // copy package.json
         if (file.indexOf('package.json') !== -1) {
             return this._copyAndCleanupPackageJson(file);
         }
 
         // copy other files
         return <Observable<any>> Observable.create((observer) => {
-            let fileDest = file;
-            if (externals && file.indexOf('src/') !== -1) {
-                fileDest = file.split('src/').pop();
-            }
             fs.stat(`${this._srcPath}${file}`, (error, stats) => {
                 if (error) {
                     console.error('doesn\'t exist on copy =>', error.message);
                 }
                 if (stats && (stats.isFile() || stats.isDirectory())) {
-                    fs.copy(`${this._srcPath}${file}`, `${this._destPath}${fileDest}`, (err) => {
+                    fs.copy(`${this._srcPath}${file}`, `${this._destPath}${file}`, (err) => {
                         if (err) {
                             console.error('copy failed =>', err.message);
                         }
@@ -78,52 +71,12 @@ class Packaging {
         });
     }
 
-    /**
-     * Function to remove original file
-     *
-     * @param file {string}
-     * @param remove {boolean}
-     *
-     * @return {Observable<any>}
-     *
-     * @private
-     */
-    private _remove(file: string, remove?: boolean): Observable<any> {
-        // remove original files
-        return <Observable<any>> Observable.create((observer) => {
-            if (remove) {
-                fs.stat(`${this._srcPath}${file}`, (error, stats) => {
-                    if (error) {
-                        console.error('doesn\'t exist on remove =>', error.message);
-                    }
-
-                    if (stats && (stats.isFile() || stats.isDirectory())) {
-                        fs.remove(`${this._srcPath}${file}`, (err) => {
-                            if (err) {
-                                console.error('remove failed =>', err.message);
-                            }
-
-                            observer.next();
-                            observer.complete();
-                        });
-                    } else {
-                        observer.next();
-                        observer.complete();
-                    }
-                });
-            } else {
-                observer.next();
-                observer.complete();
-            }
-        });
-    }
-
     /**
      * Function to cleanup package.json and _copy it to dist directory
      *
      * @param file {string}
      *
-     * @return {Observable<R>}
+     * @return {Observable<any>}
      *
      * @private
      */
@@ -171,8 +124,12 @@ class Packaging {
      * Function that _copy all files in dist directory
      */
     process() {
-        Observable.forkJoin(this._files.map((fileObject: FileObject) => this._copy(fileObject.name, fileObject.externals)
-            .flatMap(_ => this._remove(fileObject.name, fileObject.remove)))).subscribe(null, error => console.error(error));
+        Observable.forkJoin(
+            this._files.map(
+                (fileObject: FileObject) => this._copy(fileObject.name)
+            )
+        )
+            .subscribe(null, error => console.error(error));
     }
 }
 

tsconfig.build.json

@@ -1,11 +1,11 @@
 {
     "extends": "./tsconfig.json",
     "compilerOptions": {
-        "outDir": ".",
+        "outDir": "./dist/commonjs",
         "rootDir": "./src",
+        "declarationDir": "./dist",
         "types": [
             "node",
-            "moment",
             "moment-timezone"
         ]
     },
@@ -15,4 +15,4 @@
         "test",
         "tools"
     ]
-}
+}

tsconfig.json

@@ -1,11 +1,12 @@
 {
     "compilerOptions": {
         "target": "es2015",
-        "module": "umd",
+        "module": "commonjs",
         "moduleResolution": "node",
         "declaration": true,
         "noImplicitAny": false,
         "sourceMap": true,
+        "inlineSources": true,
         "emitDecoratorMetadata": true,
         "experimentalDecorators": true,
         "outDir": "./tmp",
@@ -14,9 +15,8 @@
         "typeRoots": ["./node_modules/@types"],
         "types": [
             "node",
-            "moment",
-            "moment-timezone",
-            "fs-extra"
+            "fs-extra",
+            "moment-timezone"
         ],
         "lib": [
             "dom",