1
0
mirror of https://github.com/golang/go synced 2024-11-18 09:34:53 -07:00
Commit Graph

128 Commits

Author SHA1 Message Date
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
Russ Cox
45495249e1 code review fixes
* clean up error handling: show Exception info
* white space fixes
* clean up output when creating CL
* simplify hg change command; add hg file
* fix stale cookie bug (thanks iant)
* in LoadAllCL, load each CL in a different thread,
  to parallelize the slow web fetches
* throw away support for Mercurial before version 1.3
* add @CL-number file pattern for commands like diff
* make hg sync show files being sync'ed

R=r
http://go/go-review/1016016
2009-11-01 05:49:35 -08:00
Russ Cox
439f9ca809 add codereview-login command
R=r
CC=go-dev
http://go/go-review/1012007
2009-10-22 14:14:17 -07:00
Russ Cox
eea2573032 fix handling of escaped characters like ' " & < >
R=r
http://go/go-review/1013007
2009-10-22 11:21:13 -07:00
Russ Cox
79a63728eb Initial checkin of codereview plugin.
The master copy is going to live elsewhere,
but weAPOSTROPHEll keep a copy in the Go tree so that
developers donAPOSTROPHEt have to check out two
different repositories.

R=r
CC=go-dev
http://go/go-review/1012006
2009-10-22 11:12:39 -07:00