On 2014/05/15 09:47:15, jameinel wrote: > Please take a look.
2) I'm not really sure about this. Can't think of anything very pleasant really.
3) Agreed. I like "all.go" FWIW.
7) Just one test here maybe, then?
Otherwise all looking good. Other comments following.
https://codereview.appspot.com/100460045/
« Back to merge proposal
On 2014/05/15 09:47:15, jameinel wrote:
> Please take a look.
2) I'm not really sure about this. Can't think of anything very pleasant
really.
3) Agreed. I like "all.go" FWIW.
7) Just one test here maybe, then?
Otherwise all looking good. Other comments following.
https:/ /codereview. appspot. com/100460045/