summaryrefslogtreecommitdiff
path: root/lib/codereview/codereview.py
AgeCommit message (Collapse)AuthorFilesLines
2009-11-06codereview: print gofmt message when abortingRuss Cox1-3/+3
R=r CC=vish http://go/go-review/1026013
2009-11-06make temporary google.com -> golang.org hack more sophisticatedRuss Cox1-8/+8
R=r http://go/go-review/1024013
2009-11-06require gofmt for all $GOROOT/src/.../*.go.Russ Cox1-8/+31
try harder to post messages to rietveld. R=gri, r CC=go-dev http://go/go-review/1024006
2009-11-05fix UTF-8 upload bugRuss Cox1-4/+7
TBR=r http://go/go-review/1026001
2009-11-05codereview: new commandsRuss Cox1-31/+231
* 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-05add -l flag to hg gofmt.Russ Cox1-4/+9
don't apply gofmt to non-go files during upload check. R=r http://go/go-review/1016048
2009-11-04add "hg gofmt" to run all modified files through gofmt.Russ Cox1-12/+93
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-04avoid infinite recursion in matcher.Russ Cox1-15/+79
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-04various tweaks to code review.Russ Cox1-10/+27
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-02rietveld does not like code reviews with subjects longer than 100 ↵Russ Cox1-1/+3
characters, so trim the line pulled out of the description to avoid the limit. R=r http://go/go-review/1016027
2009-11-01code review: avoid subject changes, fix http_proxy submit bugRuss Cox1-11/+25
R=r http://go/go-review/1016021
2009-11-01code review fixesRuss Cox1-188/+282
* 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-10-22add codereview-login commandRuss Cox1-0/+15
R=r CC=go-dev http://go/go-review/1012007
2009-10-22fix handling of escaped characters like ' " & < >Russ Cox1-4/+3
R=r http://go/go-review/1013007
2009-10-22Initial checkin of codereview plugin.Russ Cox1-0/+2575
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
generated by cgit v1.2.3 (git 2.39.1) at 2025-04-30 06:26:26 +0000