qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-11-12 07:10:22 -07:00
Code Releases Activity
5,888 Commits 58 Branches 402 Tags 522 MiB
bbf925aa56
Commit Graph

54 Commits

Author SHA1 Message Date
Russ Cox
bbf925aa56 codereview: avoid crash if no config 
Fixes #959.

R=r
CC=golang-dev
https://golang.org/cl/1876045
 2010-07-26 17:33:50 -07:00
Russ Cox
08e65f79c1 codereview: make 'hg submit' work with Mercurial 1.6 
R=r
CC=golang-dev
https://golang.org/cl/1841044
 2010-07-16 18:54:38 -07:00
Russ Cox
ad665e4fde codereview: don't run gofmt with an empty file list 
R=gri
CC=golang-dev
https://golang.org/cl/1678054
 2010-07-15 16:43:06 -07:00
Evan Shaw
e8fcf60093 codereview: Fix for Mercurial >= 1.6 
R=rsc, adg, rsc1
CC=golang-dev
https://golang.org/cl/1804041
 2010-07-14 17:17:04 -07:00
Russ Cox
93f614ff86 codereview: allow multiple email addresses in CONTRIBUTORS 
R=r
CC=golang-dev
https://golang.org/cl/1650041
 2010-06-30 23:34:11 -07:00
Paolo Giarrusso
3ca72458ec codereview: avoid exception in match 
R=rsc
CC=golang-dev
https://golang.org/cl/1585041
 2010-06-09 21:37:11 -07:00
Peter Williams
1d6eb74697 codereview: work better with mq 
R=rsc
CC=golang-dev
https://golang.org/cl/1140043
 2010-05-24 14:37:00 -07:00
David Symonds
ee75ffa286 codereview: Mention that deleting a CL is a use of 'hg change' in its header. 
R=rsc
CC=golang-dev
https://golang.org/cl/903043
 2010-04-10 01:53:43 -07:00
Andrew Gerrand
e678afa891 codereview.py: fix for compatibility with hg >=1.4.3 
hg.parseurl now returns a two-value tuple; codereview.py expected 3.
Changed to merely take the first return value.

R=rsc, iant
CC=golang-dev
https://golang.org/cl/223087
 2010-03-03 09:03:31 +11:00
Russ Cox
4967f857d5 codereview: make sure each mail has a reviewer 
R=r
CC=golang-dev
https://golang.org/cl/196050
 2010-01-28 12:48:21 -08:00
Russ Cox
84b044e8fb codereview: correct handling of files created with hg cp 
R=r
CC=golang-dev
https://golang.org/cl/194118
 2010-01-27 16:14:11 -08:00
Russ Cox
ff7343f8d0 codereview: accommodate recent change to ui.prompt 
Fixes #525.

R=r
CC=golang-dev
https://golang.org/cl/190044
 2010-01-20 09:49:35 -08:00
Russ Cox
84ac357c79 codereview: split name list only on commas; some user names have spaces 
R=r
CC=golang-dev
https://golang.org/cl/184069
 2010-01-13 09:09:06 -08:00
Russ Cox
752b1702d0 codereview: do not send mail when submitting for others 
rename original_author to copied_from so that it reads
better when used as a condition.

R=r
CC=golang-dev
https://golang.org/cl/185045
 2010-01-09 09:47:14 -08:00
Russ Cox
1594730461 codereview: repeated hg mail sends "PTAL" 
also send out hg mail for TBR submits

R=r
CC=golang-dev
https://golang.org/cl/183137
 2010-01-07 18:23:30 -08:00
Russ Cox
8274742393 codereview: add golang-dev@googlegroups.com 
automatically in "hg mail".
also, avoid "empty list means all modified files in client" bug

R=gri, cw
CC=golang-dev
https://golang.org/cl/174072
 2009-12-15 13:36:05 -08:00
Hector Chu
31645cc0bb Ported godefs to Windows. 
R=rsc
https://golang.org/cl/164049
 2009-12-13 12:21:44 -08:00
Russ Cox
9a86cc679a codereview: do not gofmt deleted files 
R=r
https://golang.org/cl/164083
 2009-12-03 17:23:11 -08:00
Russ Cox
69f893a127 codereview: move reviewers to cc line after submit, 
so that the issue stops showing up on the
  per-reviewer dashboard page.

R=r
https://golang.org/cl/164075
 2009-12-02 09:10:59 -08:00
Devon H. O'Dell
e9a8ab031d lxml.etree is the wrong location for this alt package 
on my linux machine this is the correct one. lxml.etree
  exists with an ElementTree class, but does not contain
  an .XML method.

R=rsc
https://golang.org/cl/163082
 2009-12-02 08:18:26 -08:00
Devon H. O'Dell
742221d20b xml.etree can also be lxml.etree (e.g. CentOS 5.4 with Python 2.4.3) 
R=rsc
CC=golang-dev
https://golang.org/cl/164053
 2009-12-02 01:16:38 -08:00
Hector Chu
cd9d72ba9e Ports of lib9, libbio and libmach to Windows. 
R=rsc
https://golang.org/cl/157159
 2009-11-30 11:53:11 -08:00
Russ Cox
be32c6aaf5 codereview: allow editing of local CL description 
between hg clpatch and hg submit.

R=r
https://golang.org/cl/162059
 2009-11-30 10:28:48 -08:00
Russ Cox
19dae0799b gofmt -r 'α[β:len(α)] -> α[β:]' -w test/bench 
except chameneosredux which i know is being edited

require gofmt for test/bench

R=r
https://golang.org/cl/157110
 2009-11-20 13:11:42 -08:00
Russ Cox
caf15f855d codereview: handle UTF-8 in message descriptions 
what a terrible way to implement strings

R=r
https://golang.org/cl/156116
 2009-11-20 11:19:16 -08:00
Russ Cox
c614ffe946 codereview: make hg sync work with Mercurial 1.4 
R=r
https://golang.org/cl/157102
 2009-11-20 00:30:38 -08:00
Russ Cox
7db2c799ce codereview: more attempts at robustness in the face of unexpected exceptions 
R=r
https://golang.org/cl/156062
 2009-11-17 23:23:18 -08:00
Russ Cox
aae0aefb70 codereview: handle spaces and other unexpected chars in nicknames 
R=r
https://golang.org/cl/157053
 2009-11-17 16:52:36 -08:00
Rob Pike
c7ec6ef859 case insensitive matching in CONTRIBUTORS file 
R=rsc
https://golang.org/cl/157043
 2009-11-17 11:40:25 -08:00
Russ Cox
60df923916 codereview: discard \r characters (sigh) 
R=r
https://golang.org/cl/157041
 2009-11-17 09:08:54 -08:00
Russ Cox
1a2418f575 codereview: add clpatch --ignore_hgpatch_errors. 
of limited utility but good for creating the metadata
  for an AUTHORS/CONTRIBUTORS change even if
  the patch doesn't apply cleanly.

R=r
https://golang.org/cl/154140
 2009-11-17 08:47:48 -08:00
Devon H. O'Dell
152bfa03d8 Pass ui into PostMessage to avoid nasty/confusing exception 
R=rsc
https://golang.org/cl/155079
 2009-11-17 08:32:23 -08:00
Russ Cox
780dbdc024 codereview: clpatch tweaks 
R=agl1
https://golang.org/cl/152114
 2009-11-12 18:37:39 -08:00
Russ Cox
44fb865a48 default to codereview.appspot.com 
R=r
http://codereview.prom.corp.google.com/1026040
 2009-11-10 15:18:40 -08:00
Russ Cox
b1a52ce4a3 make hg mail more forgiving: 
create CL first, even if gofmt is needed
or no reviewers listed.

R=r
http://go/go-review/1025025
 2009-11-08 22:13:10 -08:00
Russ Cox
830813f019 assorted cleanup 
R=r, iant
CC=go-dev
http://go/go-review/1025024
 2009-11-08 21:08:27 -08:00
Russ Cox
aa2a9e4796 another attempt at utf-8 uploads. 
the last attempt broke uploads of binary files like .png

R=r
http://go/go-review/1026025
 2009-11-08 09:49:13 -08:00
Russ Cox
e67161ee3f fix authentication in parallel LoadAllCL 
R=r
http://go/go-review/1024018
 2009-11-07 18:56:29 -08:00
Russ Cox
72a59cecb9 require Mercurial 1.3 again (how did that get dropped?) 
avoid deprecation warning about os.popen3

R=r
http://go/go-review/1026018
 2009-11-07 17:30:40 -08:00
Russ Cox
f74beebb2f codereview: print gofmt message when aborting 
R=r
CC=vish
http://go/go-review/1026013
 2009-11-06 18:40:30 -08:00
Russ Cox
88e365ce79 make temporary google.com -> golang.org hack more sophisticated 
R=r
http://go/go-review/1024013
 2009-11-06 17:02:47 -08:00
Russ Cox
043486eb3e require gofmt for all $GOROOT/src/.../*.go. 
try harder to post messages to rietveld.

R=gri, r
CC=go-dev
http://go/go-review/1024006
 2009-11-06 09:45:24 -08:00
Russ Cox
f31bc223b3 fix UTF-8 upload bug 
TBR=r
http://go/go-review/1026001
 2009-11-05 18:23:26 -08:00
Russ Cox
790c9b59d6 codereview: new commands 
* clpatch
	* download
	* submit, on behalf of clpatch

stir hgpatch to fix a few bugs

R=r
CC=go-dev
http://go/go-review/1016051
 2009-11-05 14:44:57 -08:00
Russ Cox
9df7d6ef10 add -l flag to hg gofmt. 
don't apply gofmt to non-go files during upload check.

R=r
http://go/go-review/1016048
 2009-11-05 08:11:44 -08:00
Russ Cox
d8e0d9a778 add "hg gofmt" to run all modified files through gofmt. 
add check before Upload, for now disabled by default,
that files do not change when run through gofmt.
enable by adding

	[codereview]
	force_gofmt = True

to .hgrc or .hg/hgrc.

add doc strings for a few more commands.
rename codereview-login to code-login
to make module doc visible.

R=r
CC=gri
http://go/go-review/1018056
 2009-11-04 23:43:55 -08:00
Russ Cox
e414fda6d4 avoid infinite recursion in matcher. 
after sync (or sync --local), clean up repository:
	* look for and close CLs submitted on our behalf
	* remove unmodified files from CLs
	* warn about empty CLs

R=r
http://go/go-review/1017029
 2009-11-04 15:17:01 -08:00
Russ Cox
506ce11f03 various tweaks to code review. 
main one is to check at submit time that
user name being used in checkin message
is listed in the CONTRIBUTORS file.
this should catch misconfigurations.

another is to cut the @domain part
from the R= and CC= lines on checkin
messages, so that cc'ing someone on
a change does not mean their email
address is recorded for all time.

R=r
CC=go-dev
http://go/go-review/1016036
 2009-11-04 03:15:24 -08:00
Russ Cox
9c13215859 rietveld does not like code reviews with subjects longer than 100 characters, so trim the line pulled out of the description to avoid the limit. 
R=r
http://go/go-review/1016027
 2009-11-02 11:37:21 -08:00
Russ Cox
dde666dab5 code review: avoid subject changes, fix http_proxy submit bug 
R=r
http://go/go-review/1016021
 2009-11-01 18:46:07 -08:00