Merge pull request #245 from aws-samples/devops-buildspec-npm-fix

Fix bug with npm postinstall via buildspec.yml by changing to 'npm install --prefix' instead

Author: samdengler

DevOps/2_ContinuousDeliveryPipeline/uni-api/buildspec.yml

@@ -6,6 +6,10 @@ phases:
       nodejs: 10
 
     commands:
+      # Install application dependencies
+      - npm install --prefix app
+
+      # Install dependencies needed for running tests
       - npm install
 
       # Upgrade AWS CLI to the latest version

DevOps/2_ContinuousDeliveryPipeline/uni-api/package.json

@@ -5,9 +5,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {},
-  "scripts": {
-    "postinstall": "cd app && npm install"
-  },
+  "scripts": {},
   "devDependencies": {
     "mocha": "^4.0.1",
     "proxyquire": "^1.8.0"

DevOps/3_XRay/uni-api/app/package-lock.json

@@ -5,7 +5,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "aws-xray-sdk": "^2.153.0",
+    "aws-sdk": "^2.153.0",
     "aws-xray-sdk": "^1.1.6"
   }
 }

DevOps/3_XRay/uni-api/app/package.json

@@ -6,6 +6,9 @@ phases:
       nodejs: 10
 
     commands:
+      # Install application dependencies
+      - npm install --prefix app
+
       # Install dependencies needed for running tests
       - npm install
 

DevOps/3_XRay/uni-api/buildspec.yml

@@ -5,9 +5,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {},
-  "scripts": {
-    "postinstall": "cd app && npm install"
-  },
+  "scripts": {},
   "devDependencies": {
     "mocha": "^4.0.1",
     "proxyquire": "^1.8.0"