search

justintadlock / hybrid-base

A base theme for building with the Hybrid Core framework.
GNU General Public License v2.0
154 stars 73 forks source link

Kill trailing whitespace #2

Closed lkraav closed 11 years ago

lkraav commented 11 years ago

Minor cleanup with my regular workhorse

alias find-kill-trailing-whitespace='find . -path ./.git -prune -o \( -name "*.js" -o -name "*.php" -o -name "*.css" \) -exec sed -i "s/ *$//" {} \;'

It would be awesome if the theme (or author) would drop tabs in favor of spaces (:set expandtab ftw!), as well, but I'm not counting on it :)

justintadlock commented 11 years ago

I'm just closing this as "wontfix". I honestly don't see much point in removing a few spaces.

As for dropping tabs in favor of spaces, that'll most likely never happen.

lkraav commented 11 years ago

Well not pulling it means automated tools would always want to make this change again unless specifically instructed not to. I for one have whitespace errors all highlighted in my editor and no doubt so do many other developers. Just saying. Successful automation win is a win even if it's a small one..