yellow team mailing list archive

[Kapil Thangavelu <kapil.thangavelu@xxxxxxxxxxxxx>]

addressed both, thanks.


On Sat, Oct 20, 2012 at 7:09 PM, <benjamin.saller@xxxxxxxxxxxxx> 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.