<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 6, 2014 at 10:12 AM, Vlad Dumitrescu <span dir="ltr"><<a href="mailto:vladdu55@gmail.com" target="_blank">vladdu55@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Feb 6, 2014 at 9:38 AM, Loïc Hoguin <<a href="mailto:essen@ninenines.eu">essen@ninenines.eu</a>> wrote:<br>


> The contributor can quickly see if they broke whitespace after submitting<br>
> the PR on github by looking at the diff tab. I'm always surprised when<br>
> people submit a PR with obviously broken whitespace and do nothing about it<br>
> until I point it out.<br>
<br>
</div>As an interesting anecdote, yesterday I managed somehow to produce a<br>
patch that looked good on github, but not in emacs... So unfortunately<br>
it seems that checking GH might not be enough (at least not when not<br>
using emacs in the first place).<br>
<br><br></blockquote><div>There are also differences between a `git diff` and github display sometimes... </div><div><br></div><div>For myself I am surprised by the number of project owner not describing the standard they are following but expecting that others are following it. Which create frustration on each parts.</div>

<div><br></div><div>This why the gofmt tool or pep8 (even if I hate this one) are good. They allows a lot of people using the same language to follow the same guidelines. And provides tools to automatically fix them when needed. it removes a lot of frustration from each parts.</div>

<div><br></div><div>- benoit</div></div></div></div>