<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Mar 16, 2016 at 12:48 AM, Kevin Smith <span dir="ltr"><<a href="mailto:ksmith@wikimedia.org" target="_blank">ksmith@wikimedia.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div>I would mention that in some cases, I would prefer to accept the commit as is, and then perform minor refactoring, such as changing a name, fixing a typo, or rearranging the code. Not only does that clearly separate authorship, but it would also encourage those changes to be reviewed by someone other than that author. <br></div></div></div></blockquote></div><br></div><div class="gmail_extra">Git log is the public record of what changes happened to the code, and is regularly read for various reasons (creating changelogs, identifying commits that caused regressions, looking for relevant changes that happened to master since a feature branch was split, getting a sense of recent changes). Lots of trivial formatting commits add an overhead to that. Same with git blame, which is often needed in refactorings to understand why a certain line of code is there.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I note that if someone wants to get input about the effect certain workflow changes would have on developers' productivity, wikitech-l is probably a better place to ask than teampractices.</div></div>