addressed both, thanks.
On Sat, Oct 20, 2012 at 7:09 PM, <[email protected]> wrote: > Some updates, after that lgtm > > > https://codereview.appspot.**com/6734052/diff/3001/README<https://codereview.appspot.com/6734052/diff/3001/README> > File README (left): > > https://codereview.appspot.**com/6734052/diff/3001/README#**oldcode71<https://codereview.appspot.com/6734052/diff/3001/README#oldcode71> > README:71: jshint --config=jshint.config `bzr ls -RV -k file | grep -v > assets/` > make lint > > https://codereview.appspot.**com/6734052/diff/3001/README<https://codereview.appspot.com/6734052/diff/3001/README> > File README (right): > > https://codereview.appspot.**com/6734052/diff/3001/README#**newcode25<https://codereview.appspot.com/6734052/diff/3001/README#newcode25> > README:25: bzr branch lp:juju-ui trunk > lp:juju-gui > > https://codereview.appspot.**com/6734052/<https://codereview.appspot.com/6734052/> > -- https://code.launchpad.net/~hazmat/juju-gui/readme-update/+merge/130690 Your team Juju GUI Hackers is requested to review the proposed merge of lp:~hazmat/juju-gui/readme-update into lp:juju-gui. -- Mailing list: https://launchpad.net/~yellow Post to : [email protected] Unsubscribe : https://launchpad.net/~yellow More help : https://help.launchpad.net/ListHelp
