<p dir="ltr"><br>
On 20 Jun 2015 11:50, "Martin Nowak via dmd-internals" <<a href="mailto:dmd-internals@puremagic.com">dmd-internals@puremagic.com</a>> wrote:<br>
><br>
> On 05/25/2015 10:36 PM, Walter Bright via dmd-internals wrote:<br>
> ><br>
> ><br>
> > On 5/25/2015 11:28 AM, Steven Schveighoffer via dmd-internals wrote:<br>
> >> I feel like I need to respond to this. For the most part, I think the<br>
> >> “bouncing the rubble” points are spot on. Reorganizing files can have<br>
> >> benefits, and generally github is quite good at recognizing the move.<br>
> ><br>
> > My experience with github is it is quite terrible at showing a<br>
> > reasonable diff when there's been whitespace and code motion changes.<br>
><br>
> There is a nice trick, append ?w=1 to the diff page and it will omit<br>
> whitespace changes.<br>
><br>
> <a href="https://github.com/D-Programming-Language/phobos/pull/3431/files?w=1">https://github.com/D-Programming-Language/phobos/pull/3431/files?w=1</a><br>
></p>
<p dir="ltr">And if that does not work, asking people to split up their commits into easy to review chunks helps too.</p>
<p dir="ltr"><a href="https://github.com/D-Programming-Language/dlang.org/pull/981">https://github.com/D-Programming-Language/dlang.org/pull/981</a></p>
<p dir="ltr">If it helps speed up the review / merge process... </p>