code review 12741045: go-tour: add slide about sandbox environment

go-tour: add slide about sandbox environment

Also mention rand.Seed.

Fixes issue 108.

[adg, 2013-08-14 21:13:43]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go-tour

[r, 2013-08-14 21:17:01]

https://codereview.appspot.com/12741045/diff/1/tour.article
File tour.article (right):

https://codereview.appspot.com/12741045/diff/1/tour.article#newcode69
tour.article:69: #appengine: (To see a different number, seed the number
generator with [[http://golang.org/pkg/math/rand/#Seed][`rand.Seed`]].)
the usual seed is time, and that won't work either.

[adg, 2013-08-15 03:16:09]

On 2013/08/14 21:17:01, r wrote:
> https://codereview.appspot.com/12741045/diff/1/tour.article
> File tour.article (right):
> 
> https://codereview.appspot.com/12741045/diff/1/tour.article#newcode69
> tour.article:69: #appengine: (To see a different number, seed the number
> generator with [[http://golang.org/pkg/math/rand/#Seed][`rand.Seed`]].)
> the usual seed is time, and that won't work either.

That doesn't make the text incorrect.

[r, 2013-08-15 03:17:32]

it does make it unhelpful, though

[adg, 2013-08-15 03:41:08]

Hello golang-dev@googlegroups.com, r@golang.org (cc:
golang-dev@googlegroups.com),

Please take another look.

[r, 2013-08-15 04:06:23]

LGTM

https://codereview.appspot.com/12741045/diff/8001/tour.article
File tour.article (right):

https://codereview.appspot.com/12741045/diff/8001/tour.article#newcode87
tour.article:87: #appengine: (To see a different number, seed the number
generator with [[http://golang.org/pkg/math/rand/#Seed][`rand.Seed`]].)
s/ with/; see/
or / as described here:/

[adg, 2013-08-15 04:10:56]

*** Submitted as https://code.google.com/p/go-tour/source/detail?r=3f353c59248f
***

go-tour: add slide about sandbox environment

Also mention rand.Seed.

Fixes issue 108.

R=golang-dev, r
CC=golang-dev
https://codereview.appspot.com/12741045

[kortschak, 2013-08-15 05:08:36]

NOT LGTM

prog/sandbox.go does not exist.

On 2013/08/15 04:10:56, adg wrote:
> *** Submitted as
https://code.google.com/p/go-tour/source/detail?r=3f353c59248f
> ***
> 
> go-tour: add slide about sandbox environment
> 
> Also mention rand.Seed.
> 
> Fixes issue 108.
> 
> R=golang-dev, r
> CC=golang-dev
> https://codereview.appspot.com/12741045

[adg, 2013-08-15 05:10:01]

Ah, sorry: https://codereview.appspot.com/12976043


On 15 August 2013 15:08, <dan.kortschak@adelaide.edu.au> wrote:

> NOT LGTM
>
> prog/sandbox.go does not exist.
>
>
> On 2013/08/15 04:10:56, adg wrote:
>
>> *** Submitted as
>>
>
https://code.google.com/p/go-**tour/source/detail?r=**3f353c59248f<https://co...
>
>> ***
>>
>
>  go-tour: add slide about sandbox environment
>>
>
>  Also mention rand.Seed.
>>
>
>  Fixes issue 108.
>>
>
>  R=golang-dev, r
>> CC=golang-dev
>>
https://codereview.appspot.**com/12741045<https://codereview.appspot.com/1274...
>>
>
>
>
>
https://codereview.appspot.**com/12741045/<https://codereview.appspot.com/127...
>
> --
>
> ---You received this message because you are subscribed to the Google
> Groups "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to
golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou...
> .
> For more options, visit
https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o...
> .
>