Please take a look.
https://codereview.appspot.com/12143043/diff/22001/container/lxc/lxc_test.go File container/lxc/lxc_test.go (right):
https://codereview.appspot.com/12143043/diff/22001/container/lxc/lxc_test.go#newcode128 container/lxc/lxc_test.go:128: "ifconfig"}) On 2013/08/05 17:35:39, rog wrote:
> We'd usually put the }) on a new line here, so each string gets its own line.
Done.
https://codereview.appspot.com/12143043/diff/22001/utils/apt.go File utils/apt.go (right):
https://codereview.appspot.com/12143043/diff/22001/utils/apt.go#newcode61 utils/apt.go:61: "Acquire::ftp::Proxy"} On 2013/08/05 17:35:39, rog wrote:
> newline before } please
https://codereview.appspot.com/12143043/
« Back to merge proposal
Please take a look.
https:/ /codereview. appspot. com/12143043/ diff/22001/ container/ lxc/lxc_ test.go lxc/lxc_ test.go (right):
File container/
https:/ /codereview. appspot. com/12143043/ diff/22001/ container/ lxc/lxc_ test.go# newcode128 lxc/lxc_ test.go: 128: "ifconfig"})
container/
On 2013/08/05 17:35:39, rog wrote:
> We'd usually put the }) on a new line here, so each string gets its
own line.
Done.
https:/ /codereview. appspot. com/12143043/ diff/22001/ utils/apt. go
File utils/apt.go (right):
https:/ /codereview. appspot. com/12143043/ diff/22001/ utils/apt. go#newcode61 :ftp::Proxy" }
utils/apt.go:61: "Acquire:
On 2013/08/05 17:35:39, rog wrote:
> newline before } please
Done.
https:/ /codereview. appspot. com/12143043/