Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(288)

Unified Diff: README.git-cl.md

Issue 2296393003: Proofread README.git-cl.md. (Closed)
Patch Set: Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: README.git-cl.md
diff --git a/README.git-cl.md b/README.git-cl.md
index f8ea82d080a0462ddffb4baff1bd5d74b7a4f211..f479eb9a077c90b1923619752f8f9199367a2ca8 100644
--- a/README.git-cl.md
+++ b/README.git-cl.md
@@ -6,15 +6,15 @@ git but unfamiliar with the code review process supported by Rietveld and
Gerrit.
-## Reitveld concepts and terms
+## Rietveld concepts and terms
A Rietveld review is for discussion of a single change or patch. You upload a
proposed change, the reviewer comments on your change, and then you can upload a
revised version of your change. Rietveld stores the history of uploaded patches
as well as the comments, and can compute diffs in between these patches. The
history of a patch is very much like a small branch in git, but since Rietveld
-is VCS-agnostic the concepts don't map perfectly. The identifier for a single
-review+patches+comments in Rietveld is called an `issue`.
+is VCS-agnostic, the concepts don't map perfectly. The identifier for a single
+review thread including patches and comments in Rietveld is called an **issue**.
Rietveld provides a basic uploader that understands git. This program is used by
git-cl, and is included in the git-cl repo as upload.py.
@@ -29,9 +29,9 @@ different ways you can handle this workflow with git:
1. Rewriting a single commit. Say the origin commit is O, and you commit your
initial work in a commit A, making your history like O--A. After review
- comments, you commit --amend, effectively erasing A and making a new commit
- A', so history is now O--A'. (Equivalently, you can use git reset --soft or
- git rebase -i.)
+ comments, you `git commit --amend`, effectively erasing A and making a new
+ commit A', so history is now O--A'. (Equivalently, you can use
+ `git reset --soft` or `git rebase -i`.)
2. Writing follow-up commits. Initial work is again in A, and after review
comments, you write a new commit B so your history looks like O--A--B. When
you upload the revised patch, you upload the diff of O..B, not A..B; you
@@ -78,9 +78,10 @@ that as a single review, everything works just as above.
But what if you upload each of A, B, and C as separate reviews? What if you
then need to change A?
-1. One option is rewriting history: write a new commit A', then use git rebase
- -i to insert that diff in as O--A--A'--B--C as well as squash it. This is
- sometimes not possible if B and C have touched some lines affected by A'.
+1. One option is rewriting history: write a new commit A', then use
+ `git rebase -i` to insert that diff in as O--A--A'--B--C as well as squash
+ it. This is sometimes not possible if B and C have touched some lines
+ affected by A'.
2. Another option, and the one espoused by software like topgit, is for you to
have separate branches for A, B, and C, and after writing A' you merge it
into each of those branches. (topgit automates this merging process.) This
@@ -98,15 +99,15 @@ then need to change A?
In practice, this comes up pretty rarely. Suggestions for better workflows are
welcome.
-## Bash auto complition
+## Bash auto completion
1. Ensure that your base git commands are autocompleted
[doc](https://git-scm.com/book/en/v1/Git-Basics-Tips-and-Tricks).
2. Add this to your .bashrc:
-
+
# The next line enables bash completion for git cl.
if [ -f "$HOME/bin/depot_tools/git_cl_completion.sh" ]; then
. "$HOME/bin/depot_tools/git_cl_completion.sh"
fi
-
+
3. Profit.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698