Skip to content

Commit

Permalink
Fix missing package winston
Browse files Browse the repository at this point in the history
FAB-2034
winston is required as the default logger used by both
fabric-client and fabric-ca-client. It's currently specified
as a dev dependency so if node.js app use the npm production mode
install this will not be installed.

Fix is to move this to the "for-production" dependency list

Change-Id: Ib62d90131166bdea00c94286faf3ba0af64f8dac
Signed-off-by: Jim Zhang <[email protected]>
  • Loading branch information
jimthematrix committed Feb 6, 2017
1 parent 77ff639 commit 0fd7d2c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
3 changes: 2 additions & 1 deletion fabric-ca-client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,8 @@
"sjcl-codec": "0.1.1",
"tar-fs": "^1.13.0",
"url": "^0.11.0",
"util": "^0.10.3"
"util": "^0.10.3",
"winston": "^2.2.0"
},
"license": "Apache-2.0",
"licenses": [{
Expand Down
6 changes: 3 additions & 3 deletions fabric-client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,8 @@
"tar-fs": "^1.13.0",
"url": "^0.11.0",
"util": "^0.10.3",
"uuidv4": "^0.3.1"
"uuidv4": "^0.3.1",
"winston": "^2.2.0"
},
"devDependencies": {
"bunyan": "^1.8.1",
Expand All @@ -60,8 +61,7 @@
"require-dir": "^0.3.0",
"tap-colorize": "^1.2.0",
"tape": "^4.5.1",
"tape-promise": "^1.1.0",
"winston": "^2.2.0"
"tape-promise": "^1.1.0"
},
"license": "Apache-2.0",
"licenses": [
Expand Down

0 comments on commit 0fd7d2c

Please sign in to comment.