Merge lp:~mariusko/charms/precise/node-app/trunk into lp:charms/node-app

Proposed by Marius B. Kotsbak on 2013-04-04
Status: Merged
Merged at revision: 34
Proposed branch: lp:~mariusko/charms/precise/node-app/trunk
Merge into: lp:charms/node-app
Diff against target: 11 lines (+1/-0)
1 file modified
hooks/config-changed (+1/-0)
To merge this branch: bzr merge lp:~mariusko/charms/precise/node-app/trunk
Reviewer Review Type Date Requested Status
Marco Ceppi 2013-04-04 Approve on 2013-04-15
Review via email: mp+157099@code.launchpad.net
To post a comment you must log in.
Marco Ceppi (marcoceppi) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/config-changed'
2--- hooks/config-changed 2013-03-08 13:06:46 +0000
3+++ hooks/config-changed 2013-04-04 12:53:27 +0000
4@@ -92,6 +92,7 @@
5 echo "Found new latest commit \${new_commit}"
6
7 git checkout origin/\${app_branch}
8+ npm update && npm install
9 sudo service \${app_name} restart
10 else
11 echo 'No updates found';

Subscribers

People subscribed via source and target branches