Merge #15444: [docs] Additional productivity tips
ff7f31e07d
[doc] productivity: more advanced git range-diff (Sjors Provoost)3a21905a4e
[doc] devtools: mention clang-format dependency (Sjors Provoost)bf12093191
[doc] productivity: fix broken link (Sjors Provoost) Pull request description: Fixes a broken link to `devtools/README.md`, points out the `clang-format` dependency and adds a `git range-diff` incantation that works even with rebases and squashes. Tree-SHA512: 36e46282f1e28d1bf3f48ada995fbac548f61b7747091eb032b60919cf76c7bdad0fa8aecb0c47adbdaa9ef986d3ec7752b0bb94c63191401856e2ddeec48f3e
This commit is contained in:
commit
b83c6f7940
2 changed files with 17 additions and 3 deletions
|
@ -7,6 +7,8 @@ clang-format-diff.py
|
||||||
|
|
||||||
A script to format unified git diffs according to [.clang-format](../../src/.clang-format).
|
A script to format unified git diffs according to [.clang-format](../../src/.clang-format).
|
||||||
|
|
||||||
|
Requires `clang-format`, installed e.g. via `brew install clang-format` on macOS.
|
||||||
|
|
||||||
For instance, to format the last commit with 0 lines of context,
|
For instance, to format the last commit with 0 lines of context,
|
||||||
the script should be called from the git root folder as follows.
|
the script should be called from the git root folder as follows.
|
||||||
|
|
||||||
|
|
|
@ -76,7 +76,7 @@ Writing code
|
||||||
|
|
||||||
### Format C/C++/Protobuf diffs with `clang-format-diff.py`
|
### Format C/C++/Protobuf diffs with `clang-format-diff.py`
|
||||||
|
|
||||||
See [contrib/devtools/README.md](contrib/devtools/README.md#clang-format-diff.py).
|
See [contrib/devtools/README.md](/contrib/devtools/README.md#clang-format-diff.py).
|
||||||
|
|
||||||
### Format Python diffs with `yapf-diff.py`
|
### Format Python diffs with `yapf-diff.py`
|
||||||
|
|
||||||
|
@ -136,7 +136,7 @@ This will add an `upstream-pull` remote to your git repository, which can be fet
|
||||||
|
|
||||||
### Diff the diffs with `git range-diff`
|
### Diff the diffs with `git range-diff`
|
||||||
|
|
||||||
It is very common for contributors to rebase their pull requests, or make changes to commits (perhaps in response to review) that are not at the head of their branch. This poses a problem for reviewers as when the contributor force pushes, the reviewer is no longer sure that his previous reviews of commits are still valid (as the commit hashes can now be different even though the diff is semantically the same). `git range-diff` can help solve this problem by diffing the diffs.
|
It is very common for contributors to rebase their pull requests, or make changes to commits (perhaps in response to review) that are not at the head of their branch. This poses a problem for reviewers as when the contributor force pushes, the reviewer is no longer sure that his previous reviews of commits are still valid (as the commit hashes can now be different even though the diff is semantically the same). [git range-diff](https://git-scm.com/docs/git-range-diff) (Git >= 2.19) can help solve this problem by diffing the diffs.
|
||||||
|
|
||||||
For example, to identify the differences between your previously reviewed diffs P1-5, and the new diffs P1-2,N3-4 as illustrated below:
|
For example, to identify the differences between your previously reviewed diffs P1-5, and the new diffs P1-2,N3-4 as illustrated below:
|
||||||
```
|
```
|
||||||
|
@ -152,7 +152,19 @@ You can do:
|
||||||
git range-diff master previously-reviewed-head new-head
|
git range-diff master previously-reviewed-head new-head
|
||||||
```
|
```
|
||||||
|
|
||||||
Note that `git range-diff` also work for rebases.
|
Note that `git range-diff` also work for rebases:
|
||||||
|
|
||||||
|
```
|
||||||
|
P1--P2--P3--P4--P5 <-- previously-reviewed-head
|
||||||
|
/
|
||||||
|
...--m--m1--m2--m3 <-- master
|
||||||
|
\
|
||||||
|
P1--P2--N3--N4 <-- new-head (with P3 modified, P4 & P5 squashed)
|
||||||
|
|
||||||
|
PREV=P5 N=4 && git range-diff `git merge-base --all HEAD $PREV`...$PREV HEAD~$N...HEAD
|
||||||
|
```
|
||||||
|
|
||||||
|
Where `P5` is the commit you last reviewed and `4` is the number of commits in the new version.
|
||||||
|
|
||||||
-----
|
-----
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue