Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(921)

Issue 2499041: code review 2499041: release.2010-10-13 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by adg
Modified:
12 years, 1 month ago
Reviewers:
CC:
r, rsc, r2, golang-dev
Visibility:
Public.

Description

release.2010-10-13

Patch Set 1 #

Patch Set 2 : code review 2499041: release.2010-10-13 #

Total comments: 1

Patch Set 3 : code review 2499041: release.2010-10-13 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+73 lines, -1 line) Patch
M .hgtags View 1 chunk +0 lines, -1 line 0 comments Download
M doc/devel/release.html View 1 2 1 chunk +73 lines, -0 lines 0 comments Download

Messages

Total messages: 6
adg
Hello r (cc: golang-dev@googlegroups.com), I'd like you to review this change.
12 years, 1 month ago (2010-10-13 23:42:52 UTC) #1
rsc
please resync from the docs text.
12 years, 1 month ago (2010-10-14 00:12:39 UTC) #2
adg
Hello r, rsc (cc: golang-dev@googlegroups.com), Please take another look.
12 years, 1 month ago (2010-10-14 00:18:46 UTC) #3
rsc1
LGTM after wrapping the very long line http://codereview.appspot.com/2499041/diff/3001/doc/devel/release.html File doc/devel/release.html (right): http://codereview.appspot.com/2499041/diff/3001/doc/devel/release.html#newcode32 doc/devel/release.html:32: The language ...
12 years, 1 month ago (2010-10-14 00:21:52 UTC) #4
r2
LGTM On Oct 13, 2010, at 5:21 PM, rsc@google.com wrote: > LGTM after wrapping the ...
12 years, 1 month ago (2010-10-14 00:24:55 UTC) #5
adg
12 years, 1 month ago (2010-10-14 03:52:13 UTC) #6
*** Submitted as http://code.google.com/p/go/source/detail?r=7c2e97710bf4 ***

release.2010-10-13

R=r, rsc, r2
CC=golang-dev
http://codereview.appspot.com/2499041
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b