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

Issue 4168059: code review 4168059: .hgignore: added *.cgo*.{c,go} (Closed)

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

Description

.hgignore: added *.cgo*.{c,go}

Patch Set 1 #

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

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

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M .hgignore View 1 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 5
niemeyer
Hello golang-dev@googlegroups.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
14 years ago (2011-02-21 04:11:27 UTC) #1
rsc
LGTM
14 years ago (2011-02-22 15:42:50 UTC) #2
rsc
*** Submitted as a65868992d4e *** .hgignore: add *.cgo*.{c,go} R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/4168059 Committer: Russ Cox ...
14 years ago (2011-02-22 20:42:16 UTC) #3
rsc
Applied. > .hgignore: added *.cgo*.{c,go} please use imperative, not past tense, in descriptions. so s/added/add/ ...
14 years ago (2011-02-22 20:42:49 UTC) #4
niemeyer
14 years ago (2011-02-23 02:55:00 UTC) #5
> please use imperative, not past tense, in descriptions.
> so s/added/add/ (done during submit)

Ack.
Sign in to reply to this message.

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