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

Issue 4309043: code review 4309043: doc: update contrib.html to be more enticing (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 11 months ago by adg
Modified:
13 years, 11 months ago
Reviewers:
CC:
r, rsc, golang-dev
Visibility:
Public.

Description

doc: update contrib.html to be more enticing doc: refer to golang-dev instead of -nuts in contribute.html

Patch Set 1 #

Total comments: 8

Patch Set 2 : diff -r 208240127c82 https://go.googlecode.com/hg/ #

Total comments: 3

Patch Set 3 : diff -r 31eae9ad9d23 https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r 31eae9ad9d23 https://go.googlecode.com/hg/ #

Patch Set 5 : diff -r 31eae9ad9d23 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+33 lines, -12 lines) Patch
M doc/contrib.html View 1 2 3 2 chunks +33 lines, -12 lines 0 comments Download

Messages

Total messages: 8
adg
Hello r (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
13 years, 11 months ago (2011-03-23 03:47:54 UTC) #1
r
http://codereview.appspot.com/4309043/diff/1/doc/contrib.html File doc/contrib.html (right): http://codereview.appspot.com/4309043/diff/1/doc/contrib.html#newcode14 doc/contrib.html:14: (Of course, you should search for an existing issue ...
13 years, 11 months ago (2011-03-23 03:54:49 UTC) #2
adg
http://codereview.appspot.com/4309043/diff/1/doc/contrib.html File doc/contrib.html (right): http://codereview.appspot.com/4309043/diff/1/doc/contrib.html#newcode14 doc/contrib.html:14: (Of course, you should search for an existing issue ...
13 years, 11 months ago (2011-03-23 05:24:42 UTC) #3
r
http://codereview.appspot.com/4309043/diff/4001/doc/contrib.html File doc/contrib.html (right): http://codereview.appspot.com/4309043/diff/4001/doc/contrib.html#newcode33 doc/contrib.html:33: open issues that interst you. Those labelled s/interst/interest/ as ...
13 years, 11 months ago (2011-03-23 18:26:47 UTC) #4
rsc
http://codereview.appspot.com/4309043/diff/4001/doc/contribute.html File doc/contribute.html (right): http://codereview.appspot.com/4309043/diff/4001/doc/contribute.html#newcode23 doc/contribute.html:23: mail to the <a href="http://groups.google.com/group/golang-dev">golang-dev I don't think golang-dev ...
13 years, 11 months ago (2011-03-23 18:28:19 UTC) #5
adg
On 24 March 2011 05:26, <r@golang.org> wrote: > http://codereview.appspot.com/4309043/diff/4001/doc/contrib.html#newcode33 > doc/contrib.html:33: open issues that interst ...
13 years, 11 months ago (2011-03-23 23:32:31 UTC) #6
r
LGTM
13 years, 11 months ago (2011-03-23 23:34:32 UTC) #7
adg
13 years, 11 months ago (2011-03-24 00:06:12 UTC) #8
*** Submitted as http://code.google.com/p/go/source/detail?r=88e7824cdba3 ***

doc: update contrib.html to be more enticing
doc: refer to golang-dev instead of -nuts in contribute.html

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

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