summaryrefslogtreecommitdiff
path: root/lib/codereview/codereview.py
Commit message (Collapse)AuthorAgeFilesLines
* codereview: protect against read-only upstream repositoryShenghou Ma2012-10-201-1/+2
| | | | | | R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/6742053
* codereview.py: Fix crash on auth error handling.Uriel Mangado2012-10-111-9/+11
| | | | | | | | | | | | In recent Python versions .reason is a read-only property that simply gives you the msg value. Fixes issue 4024 R=golang-dev, adg CC=golang-dev http://codereview.appspot.com/6545052 Committer: Shenghou Ma <minux.ma@gmail.com>
* codereview: more mercurial interface fixesShenghou Ma2012-10-081-14/+14
| | | | | | | | Fixes issue 4131. R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/6614061
* codereview.py: suggest installing Mercurial from official websiteRobert Hencke2012-10-031-5/+1
| | | | | | | | | | Continuation of http://codereview.appspot.com/6499053/ R=golang-dev, minux.ma CC=golang-dev http://codereview.appspot.com/6584059 Committer: Rob Pike <r@golang.org>
* codereview: use subprocess.call() instead of os.spawnvp() for portabilityShivakumar GN2012-09-271-1/+1
| | | | | | | | | | Fixes issue 4121. R=golang-dev, minux.ma, dave, r CC=golang-dev http://codereview.appspot.com/6555049 Committer: Shenghou Ma <minux.ma@gmail.com>
* codereview.py: correct error handling without decoratorUriel Mangado2012-09-011-45/+32
| | | | | | | | | | | | | | | | The decorator hides the number of function arguments from Mercurial, so Mercurial cannot give proper error messages about commands invoked with the wrong number of arguments. Left a 'dummy' hgcommand decorator in place as a way to document what functions are hg commands, and just in case we need some other kind of hack in the future. R=adg, rsc CC=golang-dev http://codereview.appspot.com/6488059 Committer: Russ Cox <rsc@golang.org>
* codereview: support mercurial 2.2.1Shenghou Ma2012-05-171-0/+4
| | | | | | | | | | We explicitly use plainformatter to avoid the user's debug setting changing our behavior. Fixes issue 3603. R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/6201069
* codereview: look for new release branchesRuss Cox2012-04-251-6/+5
| | | | | | | | R=adg CC=golang-dev http://codereview.appspot.com/6127048 Committer: Andrew Gerrand <adg@golang.org>
* codereview: restore help messagesAnthony Martin2012-04-171-0/+1
| | | | | | | | | Docstrings were not being set for the wrapper functions returned by the hgcommand decorator. R=golang-dev, minux.ma, rsc CC=golang-dev http://codereview.appspot.com/6059043
* codereview: give advice on 'cannot create CL outside default branch'Russ Cox2012-04-021-1/+1
| | | | | | | | Fixes issue 3430. R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/5981047
* codereview: fix for Mercurial 2.1Russ Cox2012-03-121-0/+33
| | | | | | | | | Mercurial: the Python of version control systems. Python: the Mercurial of programming languages. R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/5777066
* codereview: add submit message support for newer clone URL and subreposShenghou Ma2012-02-211-2/+10
| | | | | | | | R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/5689063 Committer: Russ Cox <rsc@golang.org>
* codereview: allow spaces in y.tab.[ch]Russ Cox2012-02-031-1/+1
| | | | | | R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/5620053
* codereview: explain how to get hgpatch in error messageBrad Fitzpatrick2012-02-021-1/+1
| | | | | | R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/5623045
* codereview: don't check default paths when codereview disabledAndrew Gerrand2012-02-021-5/+5
| | | | | | | | R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/5622043 Committer: Russ Cox <rsc@golang.org>
* codereview: ignore test files during 'hg gofmt'Russ Cox2012-01-301-2/+5
| | | | | | R=golang-dev, gri CC=golang-dev http://codereview.appspot.com/5581047
* codereview: fix initialization checkRuss Cox2012-01-291-1/+4
| | | | | | R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5596046
* codereview: die if initialized twiceRuss Cox2012-01-291-0/+11
| | | | | | | | | | If this happens, something is misconfigured. If we don't test for this explicitly, MatchAt ends up calling itself recursively forever. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5576066
* codereview: more fixesRuss Cox2012-01-251-4/+3
| | | | | | | | Python and Mercurial are a matched pair. R=adg CC=golang-dev http://codereview.appspot.com/5570065
* codereview: support for subrepositoriesRuss Cox2012-01-251-18/+43
| | | | | | R=golang-dev, adg CC=golang-dev http://codereview.appspot.com/5564054
* codereview: Initialize "found" in codereview.py.Miki Tebeka2011-12-161-0/+1
| | | | | | | | | | Fixes issue 2569 (hg undo crashes when CL not found). R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/5489052 Committer: Russ Cox <rsc@golang.org>
* codereview: fix path slash issue.Yasuhiro Matsumoto2011-12-131-6/+11
| | | | | | | | R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/5487057 Committer: Russ Cox <rsc@golang.org>
* codereview: gofmt check for non-src/ filesDavid Crawshaw2011-11-181-1/+1
| | | | | | | | | | | | Gives other projects the benefit of the gofmt presubmit. Results in minor changes in the doc/ directory: find doc -name \*.go -exec gofmt -d {} \; R=rsc, gri CC=golang-dev http://codereview.appspot.com/5158042 Committer: Robert Griesemer <gri@golang.org>
* codereview: put message back in post-commit mailsRuss Cox2011-11-181-1/+1
| | | | | | R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5417049
* codereview: undo last night's rollbackRuss Cox2011-11-181-518/+627
| | | | | | | | Update version check from 1.4 to 1.9. Suggest 2.0. R=golang-dev, r, r CC=golang-dev http://codereview.appspot.com/5413046
* undo CL 5395044 / d88e746d3785Andrew Gerrand2011-11-181-620/+518
| | | | | | | | | | | | | | | | Breaks with hg 1.6.4. Roll back until it's fixed. ??? original CL description codereview: cleanup + basic tests R=adg, bradfitz CC=golang-dev http://codereview.appspot.com/5395044 ??? R=golang-dev CC=golang-dev http://codereview.appspot.com/5410047
* codereview: cleanup + basic testsRuss Cox2011-11-181-518/+620
| | | | | | R=adg, bradfitz CC=golang-dev http://codereview.appspot.com/5395044
* codereview: show LGTMs in hg pRuss Cox2011-10-191-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Shows first line of any response that the codereview server has identified as an LGTM. Example output below. 5305046: big: update for fmt interface changes Nothing terribly interesting here. Reviewer: gri@golang.org gri: LGTM CC: golang-dev@googlegroups.com Files: src/pkg/big/int.go src/pkg/big/nat.go src/pkg/big/nat_test.go src/pkg/big/rat.go 5307044: exp/template/html: use rune Nothing terribly interesting here. Reviewer: mikesamuel@gmail.com, nigeltao@golang.org mikesamuel: I don't see a type def for rune. Assuming that's a new intrinsic, LGTM. CC: golang-dev@googlegroups.com Files: src/pkg/exp/template/html/css.go src/pkg/exp/template/html/css_test.go src/pkg/exp/template/html/html.go src/pkg/exp/template/html/js.go R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5297045
* codereview: fix for Mercurial 1.9.2Russ Cox2011-10-051-16/+36
| | | | | | R=golang-dev, adg CC=golang-dev http://codereview.appspot.com/5127052
* codereview: fix hg change in Windows consoleYasuhiro Matsumoto2011-10-051-1/+3
| | | | | | | | | | | | | | | lib/codereview: Unable to use vim for 'hg change' from windows console reload(sys) break workaround for windows. see: http://mercurial.selenic.com/bts/issue2888 http://mercurial.selenic.com/bts/issue1452 Also does not work with backslash paths. R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/5150054 Committer: Russ Cox <rsc@golang.org>
* codereview: extra repo sanity checkRuss Cox2011-09-261-16/+38
| | | | | | | | | | | | Also work around Mercurial issue 3023. If anyone has local changes in their repo (due to patch queues or whatever) stop them from leaking into the main repository. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/5144043
* codereview: save CL messages in $(hg root)/last-changeRuss Cox2011-09-201-0/+12
| | | | | | | | Fixes issue 2279. R=bradfitz, r, r CC=golang-dev http://codereview.appspot.com/5096042
* codereview: Mercurial 1.9 fix for hg diff @nnnRuss Cox2011-09-081-6/+9
| | | | | | R=golang-dev, adg CC=golang-dev http://codereview.appspot.com/4969063
* codereview: set status while committing CLRuss Cox2011-08-101-0/+1
| | | | | | | | | Not sure how to detect read-only but at least this pins the blame on the hg server. R=r, bradfitz CC=golang-dev http://codereview.appspot.com/4847053
* codereview: fix for Mercurial 1.9Andrew Gerrand2011-07-111-8/+18
| | | | | | | | R=golang-dev, mirtchovski, mikioh.mikioh CC=golang-dev http://codereview.appspot.com/4686049 Committer: Russ Cox <rsc@golang.org>
* undo CL 4675045 / 0e11e08951f7Andrew Gerrand2011-07-121-19/+7
| | | | | | | | | | | | | | | | | | Breaks Mercurial 1.8.3. ??? original CL description codereview: fix for Mercurial 1.9 Fixes issue 2038. R=rsc, bobbypowers CC=golang-dev http://codereview.appspot.com/4675045 ??? R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/4685055
* codereview: fix for Mercurial 1.9Andrew Gerrand2011-07-121-7/+19
| | | | | | | | Fixes issue 2038. R=rsc, bobbypowers CC=golang-dev http://codereview.appspot.com/4675045
* codereview: restrict sync to default branchRuss Cox2011-06-291-2/+2
| | | | | | | | | 'default' is the name of the main branch, the one that isn't a release branch. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/4636072
* codereview: make --ignore_hgpatch_failure work againRuss Cox2011-06-271-4/+6
| | | | | | R=golang-dev, bradfitz CC=golang-dev http://codereview.appspot.com/4627067
* codereview.py: note that hg change -d abandons a change listRobert Hencke2011-06-201-1/+1
| | | | | | | | R=golang-dev, rsc CC=golang-dev http://codereview.appspot.com/4644043 Committer: Russ Cox <rsc@golang.org>
* codereview: handle 'null as missing field' in rietveld jsonRuss Cox2011-05-131-5/+15
| | | | | | R=golang-dev, r CC=golang-dev http://codereview.appspot.com/4543046
* codereview: fix clpatchRuss Cox2011-05-121-3/+6
| | | | | | TBR=gri CC=golang-dev http://codereview.appspot.com/4534052
* codereview: fetch metadata using JSON API, not XML scrapingRuss Cox2011-05-111-99/+82
| | | | | | | | Fixes hg clpatch. R=golang-dev, r, r CC=golang-dev http://codereview.appspot.com/4524045
* codereview: add release branch supportRuss Cox2011-05-031-13/+127
| | | | | | | | | | | | | Note that if you are working on the upcoming release branch you have to point your extension path to a copy of lib/codereview/codereview.py that won't change as the repository flips between release-branch and default branch. This warning should only apply to this one branch and only to rsc, but you never know. R=adg CC=golang-dev http://codereview.appspot.com/4446076
* codereview: various fixesRuss Cox2011-04-251-3/+18
| | | | | | | | | | Set mailed bit correctly for self-clpatch. Use repo.rollback correctly. Allow leading spaces in some C code. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/4438064
* codereview: add 'hg undo' commandRuss Cox2011-04-171-25/+143
| | | | | | R=adg, r CC=golang-dev http://codereview.appspot.com/4423045
* codereview: automatically port old diffs forwardRuss Cox2011-04-081-14/+100
| | | | | | | | | | | | | | | | | | | | | In the current codereview, if a patch was written against a version of a file that had subsequently been edited, hg clpatch would fail, even if the patch and the edits were in different parts of the file. In this situation the reviewer typically wrote back saying "please hg sync and hg mail to update the patch". This change rewrites the patch automatically, using the same transformation that hg sync + hg mail would. If the interim changes (since the patch was created) affect the same line ranges as the patch, clpatch will still refuse to apply it. But this CL should make of the trivial conflicts we see just go away. R=golang-dev, r CC=golang-dev http://codereview.appspot.com/4377046
* codereview: fix clpatchRuss Cox2011-04-071-5/+5
| | | | | | | | I love Python. Write once, crash anywhere. TBR=r CC=golang-dev http://codereview.appspot.com/4382043
* codereview: drop Author: line on self-clpatchRuss Cox2011-04-071-2/+4
| | | | | | | | | | | | | | | | | | | | | Right now if a Go developer makes a patch on one machine and then clpatches it onto another machine, changes subsequently made to the description are kept only locally, under the assumption that you are running clpatch because someone else wrote the CL, so you don't have permission to update the web. This change makes clpatch discard the "this was a clpatch" information from the metadata when you clpatch your own CLs from one machine to another. This should eliminate some confusion (for example in CL 4314054) but will no doubt introduce other confusion. R=golang-dev, r2 CC=golang-dev http://codereview.appspot.com/4387041
* codereview: recognize code URL without trailing slashRuss Cox2011-04-061-1/+1
| | | | | | R=adg CC=golang-dev http://codereview.appspot.com/4385042
View Lorry Controller Status