code review 4981047: doc: release.r60

doc: release.r60

[adg, 2011-09-07 03:36:04]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://go.googlecode.com/hg/

[dsymonds, 2011-09-07 03:39:18]

LGTM

[r, 2011-09-07 18:16:01]

LGTM

[rsc, 2011-09-07 18:39:57]

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html
File doc/devel/release.html (right):

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode33
doc/devel/release.html:33: is another "if". Since gofmt always puts those braces
in anyway, programs will
Since gofmt always puts those braces in anyway, gofmt-formatted
programs will not be affected.  To fix other programs, run gofmt.

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode68
doc/devel/release.html:68: The <code>old/template</code> package is deprecated
and will be removed at some
s/ at some point in the future// (says 'will be' already)

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode77
doc/devel/release.html:77: Uses of <a href="/pkg/container/vector/">package
container/vector</a> in the
Is this a publicly visible change?  I don't think so.
If not, can delete this paragraph.

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode407
doc/devel/release.html:407: <h3 id="r57.tool">Tools</h3>
s/.tool/.cmd/
to match rest of file

[adg, 2011-09-07 22:34:27]

PTAL

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html
File doc/devel/release.html (right):

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode33
doc/devel/release.html:33: is another "if". Since gofmt always puts those braces
in anyway, programs will
On 2011/09/07 18:39:57, rsc wrote:
> Since gofmt always puts those braces in anyway, gofmt-formatted
> programs will not be affected.  To fix other programs, run gofmt.

Done.

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode68
doc/devel/release.html:68: The <code>old/template</code> package is deprecated
and will be removed at some
On 2011/09/07 18:39:57, rsc wrote:
> s/ at some point in the future// (says 'will be' already)
> 

Done.

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode77
doc/devel/release.html:77: Uses of <a href="/pkg/container/vector/">package
container/vector</a> in the
On 2011/09/07 18:39:57, rsc wrote:
> Is this a publicly visible change?  I don't think so.
> If not, can delete this paragraph.

Done.

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode77
doc/devel/release.html:77: Uses of <a href="/pkg/container/vector/">package
container/vector</a> in the
On 2011/09/07 18:39:57, rsc wrote:
> Is this a publicly visible change?  I don't think so.
> If not, can delete this paragraph.

Done.

http://codereview.appspot.com/4981047/diff/1/doc/devel/release.html#newcode407
doc/devel/release.html:407: <h3 id="r57.tool">Tools</h3>
On 2011/09/07 18:39:57, rsc wrote:
> s/.tool/.cmd/
> to match rest of file

Done.

[adg, 2011-09-08 01:16:36]

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

Please take another look.

[rsc, 2011-09-08 01:59:51]

LGTM

[adg, 2011-09-08 02:08:15]

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

doc: release.r60

R=dsymonds, r, rsc
CC=golang-dev
http://codereview.appspot.com/4981047