Re:Fira Sans: a Free, Open Source Typeface Commissioned by M...
by nitrogen on Saturday May 24, 2014 @05:01AM
(#7792971)
Realigning the values causes all of the lines to change in a line-by-line diff (e.g. as displayed by git diff). It also makes "git blame" less helpful, since every realigned line appears to originate from the alignment commit.
[ Parent
| on HN ]
Re:Fira Sans: a Free, Open Source Typeface Commissioned by M...
by e12e on Saturday May 24, 2014 @05:44AM
(#7793010)
I agree -- but I'd think the work-around would be to do whitespace-insensitvie diffs? Which would probably be a good idea if the codebase uses indentation like this?
See eg: https://stackoverflow.com/questions/7310033/how-to-make-git-...
Re:Fira Sans: a Free, Open Source Typeface Commissioned by M...
by jacobparker on Saturday May 24, 2014 @09:25AM
(#7793280)
`git blame -w` to ignore whitespace. There are other good options (like `-M`, ignore moved lines) available in the documentation.
Re:Fira Sans: a Free, Open Source Typeface Commissioned by M...
by alxndr on Saturday May 24, 2014 @02:37PM
(#7793998)
I really wish GitHub would show me all diffs with `-w` by default, with a switch to turn it off...