-
-
-
-
-
Might read better if rephrased something like "Do not put anything after the <code>=</code> so that Mercurial will look in the default places for the extensions."
-
-
-
-
The comma here isn't strictly necessary. Not opening an issue here because I can't decide if it reads better with or without the comma.
-
I'd suggest something more like:
Once you have an account and have logged into our Review Board site, you can begin using RBTools. In your shell, navigate to a Mercurial clone of one of the Pidgin or purple-related projects, then run the <code>rbt login</code> command. You should only need to do this once, unless you change your password or have run the <code>rbt logout</code> command.
-
-
-
-
Update all of the development documentation
Review Request #404 — Created Jan. 5, 2021 and submitted
Add documentation for using our repos and how to contribute to them.
Built the docs locally and verified they look decent.
| Description | From | Last Updated |
|---|---|---|
|
Change "setup" to "set up" (semantics of how it's used here). |
 rekkanoryo
|
|
|
Lose the "so" at the end of this line. Reads better. |
rekkanoryo
|
|
|
Might read better if rephrased something like "Do not put anything after the <code>=</code> so that Mercurial will look in … |
rekkanoryo
|
|
|
Change "as well as" to "and". |
rekkanoryo
|
|
|
Lose the comma here. |
rekkanoryo
|
|
|
Should be "Review Board" for consistency with the rest of the text. |
rekkanoryo
|
|
|
I'd suggest something more like: Once you have an account and have logged into our Review Board site, you can … |
rekkanoryo
|
|
|
Lose the comma |
rekkanoryo
|
|
|
Do you actually mean to say dry-run here? The rest of this paragraph talks about the actual rebase. |
rekkanoryo
|
|
|
Lose "back" |
rekkanoryo
|
|
|
Lose the second "just" |
rekkanoryo
|
|
|
I missed this in the initial review, but the comma after "branching point" shouldn't be here either. |
rekkanoryo
|
