← Back to team overview

yellow team mailing list archive

Re: Updated the charm to serve Juju GUI releases. (issue 6977043)

 

https://codereview.appspot.com/6977043/diff/8001/hooks/config-changed
File hooks/config-changed (right):

https://codereview.appspot.com/6977043/diff/8001/hooks/config-changed#newcode50
hooks/config-changed:50: # Restarting of the gui and api services is
handled below.
On 2012/12/20 13:38:35, teknico wrote:

> Yes, it preexisted, but it was placed *after* a code block, making its
> interpretation less ambiguous.

Ok, replacing this comment with the one Gary suggested.

https://codereview.appspot.com/6977043/

-- 
https://code.launchpad.net/~frankban/charms/precise/juju-gui/bug-1088618-serve-releases/+merge/140745
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~frankban/charms/precise/juju-gui/bug-1088618-serve-releases into lp:~juju-gui/charms/precise/juju-gui/trunk.


References