Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed trailing whitespace. #49

Merged
merged 1 commit into from
Dec 17, 2014
Merged

Removed trailing whitespace. #49

merged 1 commit into from
Dec 17, 2014

Conversation

DrewRWx
Copy link
Member

@DrewRWx DrewRWx commented Nov 18, 2014

A nuclear option to avoid muddling good commits.

@mantaraya36
Copy link
Contributor

I would say merge this change. There are ways of asking diff programs to ignore white space.

@DrewRWx DrewRWx force-pushed the remove_whitespace branch 2 times, most recently from 9ecd633 to 7e534b9 Compare December 17, 2014 09:33
@DrewRWx
Copy link
Member Author

DrewRWx commented Dec 17, 2014

I found a tool called ack that made this a lot safer.

  • ack --print0 -l '[ \t]+$' | xargs -0 -n1 perl -pi -e 's/[ \t]+$//' killed the trailing whitespace.
  • git status --porcelain | cut -c 4- | perl -ne 'print $1 if m/\.([^.\/]+)$/' | sort -u verified the filetypes affected.

DrewRWx added a commit that referenced this pull request Dec 17, 2014
Removed trailing whitespace to save future commits from being muddled.
@DrewRWx DrewRWx merged commit 3767904 into devel Dec 17, 2014
@DrewRWx DrewRWx deleted the remove_whitespace branch December 17, 2014 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants