Whitespace
Hey everyone,
as you might remember, we recently had the discussion wether we should remove whitespace
on empty lines or keep it. This email is <b>not</b> about deciding to go for one of them, but about something
that hurts much more:
In the past, there have been many commits that actually consisted of the following changes:
1) Changes to non-empty lines (normal changes, additions, removals)
2) Adding/removing whitespace on lines, without modifying something else on that particular line.
Whenever such a commit happens, it becomes increasingly difficult, if not impossible, to digg
through the changes that actually happened, because in a merge tool every line is marked
yellow, even the one's that only have whitespace changes.
So as general rule, <b>please never commit something that consists of modified lines that only have
whitespace changes</b>.
Thanks a lot for your cooperation,
Paul<br clear="all">
--
Paul Bakaus
UI Architect
--
<a href="http://paulbakaus.com">http://paulbakaus.com</a>
<a href="http://www.linkedin.com/in/paulbakaus">http://www.linkedin.com/in/paulbakaus</a>
as you might remember, we recently had the discussion wether we should remove whitespace
on empty lines or keep it. This email is <b>not</b> about deciding to go for one of them, but about something
that hurts much more:
In the past, there have been many commits that actually consisted of the following changes:
1) Changes to non-empty lines (normal changes, additions, removals)
2) Adding/removing whitespace on lines, without modifying something else on that particular line.
Whenever such a commit happens, it becomes increasingly difficult, if not impossible, to digg
through the changes that actually happened, because in a merge tool every line is marked
yellow, even the one's that only have whitespace changes.
So as general rule, <b>please never commit something that consists of modified lines that only have
whitespace changes</b>.
Thanks a lot for your cooperation,
Paul<br clear="all">
--
Paul Bakaus
UI Architect
--
<a href="http://paulbakaus.com">http://paulbakaus.com</a>
<a href="http://www.linkedin.com/in/paulbakaus">http://www.linkedin.com/in/paulbakaus</a>
Topic Participants
Paul Bakaus
eduardolundgren
Richard Worth