1 0e9f3966 2009-08-08 rsc .TH CODEREVIEW 1
 2 0e9f3966 2009-08-08 rsc .SH NAME
 3 0e9f3966 2009-08-08 rsc codereview \- review of submitted changes (experimental)
 4 0e9f3966 2009-08-08 rsc .SH SYNOPSIS
 5 0e9f3966 2009-08-08 rsc .B hg
 6 0e9f3966 2009-08-08 rsc .B diff
 7 0e9f3966 2009-08-08 rsc .I path ...
 8 0e9f3966 2009-08-08 rsc .PP
 9 0e9f3966 2009-08-08 rsc .B codereview
10 0e9f3966 2009-08-08 rsc .I path ...
11 0e9f3966 2009-08-08 rsc .PP
12 0e9f3966 2009-08-08 rsc .B codereview
13 0e9f3966 2009-08-08 rsc .B -i
14 0e9f3966 2009-08-08 rsc .I issue
15 0e9f3966 2009-08-08 rsc .I path ...
16 0e9f3966 2009-08-08 rsc .SH DESCRIPTION
17 0e9f3966 2009-08-08 rsc .I Codereview
18 0e9f3966 2009-08-08 rsc uploads suggested changes to the code review server
19 0e9f3966 2009-08-08 rsc .I codereview.appspot.com
20 0e9f3966 2009-08-08 rsc and sends email to reviewers.
21 0e9f3966 2009-08-08 rsc It must be used from within a Plan 9 from User Space tree
22 0e9f3966 2009-08-08 rsc checked out via Mercurial
23 0e9f3966 2009-08-08 rsc (see
24 0e9f3966 2009-08-08 rsc .IR hg (1)).
25 0e9f3966 2009-08-08 rsc .PP
26 0e9f3966 2009-08-08 rsc A developer makes changes to a local copy of the tree
27 0e9f3966 2009-08-08 rsc and reviews them locally using 
28 0e9f3966 2009-08-08 rsc .I hg
29 0e9f3966 2009-08-08 rsc .I diff .
30 0e9f3966 2009-08-08 rsc When the changes are ready for external review, the 
31 0e9f3966 2009-08-08 rsc developer creates a review issue by running
32 0e9f3966 2009-08-08 rsc .I codereview
33 0e9f3966 2009-08-08 rsc naming the files to be reviewed.
34 0e9f3966 2009-08-08 rsc .I Codereview
35 0e9f3966 2009-08-08 rsc prompts for a Google Accounts user name
36 0e9f3966 2009-08-08 rsc and password to log in to 
37 0e9f3966 2009-08-08 rsc .I http://codereview.appspot.com/
38 0e9f3966 2009-08-08 rsc and create a new review issue.
39 0e9f3966 2009-08-08 rsc Once created, it prints a line like
40 0e9f3966 2009-08-08 rsc .EX
41 0e9f3966 2009-08-08 rsc Issue created. URL: http://codereview.appspot.com/96161
42 0e9f3966 2009-08-08 rsc .EE
43 0e9f3966 2009-08-08 rsc among other output.
44 0e9f3966 2009-08-08 rsc .PP
45 0e9f3966 2009-08-08 rsc The reviewer will likely respond with comments
46 0e9f3966 2009-08-08 rsc and suggestions for improving the submission.
47 0e9f3966 2009-08-08 rsc After making the changes, reupload by repeating
48 0e9f3966 2009-08-08 rsc the 
49 0e9f3966 2009-08-08 rsc .I codereview
50 0e9f3966 2009-08-08 rsc command with the
51 0e9f3966 2009-08-08 rsc .B -i
52 0e9f3966 2009-08-08 rsc option to specify the issue number
53 0e9f3966 2009-08-08 rsc (in the above example, 96161).
54 0e9f3966 2009-08-08 rsc .PP
55 0e9f3966 2009-08-08 rsc Once the reviewer accepts the changes, they will be applied
56 0e9f3966 2009-08-08 rsc automatically using the patch uploaded to the review server.
57 0e9f3966 2009-08-08 rsc To update the local tree, run 
58 0e9f3966 2009-08-08 rsc .I hg
59 0e9f3966 2009-08-08 rsc .I pull
60 0e9f3966 2009-08-08 rsc with the
61 0e9f3966 2009-08-08 rsc .B -u
62 0e9f3966 2009-08-08 rsc option.
63 0e9f3966 2009-08-08 rsc .SH SEE ALSO
64 0e9f3966 2009-08-08 rsc .HR http://codereview.appspot.com/
65 0e9f3966 2009-08-08 rsc .SH BUGS
66 0e9f3966 2009-08-08 rsc The
67 0e9f3966 2009-08-08 rsc .I codereview
68 0e9f3966 2009-08-08 rsc command should be replaced by a proper Mercurial extension.
69 0e9f3966 2009-08-08 rsc .PP
70 0e9f3966 2009-08-08 rsc Making changes to the repository via patches
71 0e9f3966 2009-08-08 rsc discards the executable bit on new shell scripts.