From 5fa8a61a238c08bca12da272f71692f2f103b6d8 Mon Sep 17 00:00:00 2001 From: Timothee Cour Date: Sun, 27 Jan 2019 22:10:20 -0800 Subject: add git diff hints to help with reviews (#10435) --- doc/contributing.rst | 20 ++++++++++++++++++++ 1 file changed, 20 insertions(+) diff --git a/doc/contributing.rst b/doc/contributing.rst index b89102a3b..3edf8352c 100644 --- a/doc/contributing.rst +++ b/doc/contributing.rst @@ -410,6 +410,26 @@ Code reviews saves time explaining the change or applying it; see also https://forum.nim-lang.org/t/4317 +2. When reviewing large diffs that may involve code moving around, github's interface + doesn't help much as it doesn't highlight moves. Instead you can use something + like this, see visual results `here `_: + + .. code-block:: sh + + git fetch origin pull/10431/head && git checkout FETCH_HEAD + git show --color-moved-ws=allow-indentation-change --color-moved=blocks HEAD^ + +3. In addition, you can view github-like diffs locally to identify what was changed + within a code block using `diff-highlight` or `diff-so-fancy`, eg: + + .. code-block:: sh + + # put this in ~/.gitconfig: + [core] + pager = "diff-so-fancy | less -R" # or: use: `diff-highlight` + + + .. include:: docstyle.rst -- cgit 1.4.1-2-gfad0