Re: [PATCH v3 1/3] merge: update documentation for {merge,diff}.renameLimit

2018-04-26 Thread Jonathan Tan
On Thu, 26 Apr 2018 16:11:50 -0700 Elijah Newren wrote: > Patch looks fine, but it's hard for me not to notice a separate issue > in this area independent of your series: I'm curious if we should > document that the value of 0 is special here (as per Jonathan Tan's > commit

Re: [PATCH v3 1/3] merge: update documentation for {merge,diff}.renameLimit

2018-04-26 Thread Elijah Newren
On Thu, Apr 26, 2018 at 1:52 PM, Ben Peart wrote: > Update the documentation to better indicate that the renameLimit setting is > ignored if rename detection is turned off via command line options or config > settings. > > Signed-off-by: Ben Peart

[PATCH v3 1/3] merge: update documentation for {merge,diff}.renameLimit

2018-04-26 Thread Ben Peart
Update the documentation to better indicate that the renameLimit setting is ignored if rename detection is turned off via command line options or config settings. Signed-off-by: Ben Peart --- Documentation/diff-config.txt | 3 ++- Documentation/merge-config.txt | 3 ++-