code review 1309045: golang.org: added Resources page, re-organized left han...

golang.org: added Resources page, re-organized left hand panel.

[adg, 2010-05-27 03:27:10]

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

I'd like you to review this change.

[r2, 2010-05-27 03:29:56]

do you have a demo site?

-rob

On May 26, 2010, at 8:27 PM, adg@golang.org wrote:

> Reviewers: r,
>
> Message:
> Hello r (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> golang.org: added Resources page, re-organized left hand panel.
>
> Please review this at http://codereview.appspot.com/1309045/show
>
> Affected files:
>  A doc/go_learning.html
>  M lib/godoc/godoc.html
>
>

[adg, 2010-05-27 23:23:41]

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

I'd like you to review this change.

[r, 2010-05-27 23:28:34]

http://codereview.appspot.com/1309045/diff/7001/8001
File doc/go_learning.html (right):

http://codereview.appspot.com/1309045/diff/7001/8001#newcode6
doc/go_learning.html:6: If you're a Go first-timer, we recommend following the
If you're new to Go (avoid colloquialisms)

http://codereview.appspot.com/1309045/diff/7001/8001#newcode7
doc/go_learning.html:7: <a href="go_tutorial.html">tutorial</a> and whilst
consulting the
s/and whilst/while/

http://codereview.appspot.com/1309045/diff/7001/8001#newcode18
doc/go_learning.html:18: The built-in docs for the Go standard library.
s/docs/documents/g

http://codereview.appspot.com/1309045/diff/7001/8001#newcode28
doc/go_learning.html:28: The official Go Language Specification. Worth reading.
It's not as heavy-going
s/S/s/

drop second and third sentences.

http://codereview.appspot.com/1309045/diff/7001/8001#newcode34
doc/go_learning.html:34: A document which specifies the conditions under which
reads of a variable in
s/which/that/

http://codereview.appspot.com/1309045/diff/7001/8001#newcode44
doc/go_learning.html:44: concepts: syntax, types, allocation, constants, IO,
sorting, printing,
s|IO|I/O|

http://codereview.appspot.com/1309045/diff/7001/8001#newcode58
doc/go_learning.html:58: application. It touches on structs, methods, file IO,
http, regular expressions,
s|IO|I/O|

http://codereview.appspot.com/1309045/diff/7001/8001#newcode64
doc/go_learning.html:64: An introduction to Go oriented toward C++ programmers.
s/oriented toward/for/

http://codereview.appspot.com/1309045/diff/7001/8001#newcode111
doc/go_learning.html:111: The langauge's first public introduction. (See the <a
href="talks/go_talk-20091030.pdf">slides in PDF format</a>.)
s/langauge/language/

The language has changed a little since it was made, but it's still a good
introduction.

http://codereview.appspot.com/1309045/diff/7001/8001#newcode128
doc/go_learning.html:128: <h2 id="videos">Blog Posts</h2>
tag looks wrong

[adg, 2010-05-27 23:37:24]

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

Please take another look.

[r, 2010-05-27 23:43:06]

LGTM

[adg, 2010-05-27 23:47:48]

*** Submitted as http://code.google.com/p/go/source/detail?r=483bf67a26c5 ***

golang.org: added Resources page, re-organized left hand panel.

R=r
CC=golang-dev
http://codereview.appspot.com/1309045