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

Cleanup trailing whitespace #360

Merged
merged 1 commit into from Feb 19, 2013
Merged

Cleanup trailing whitespace #360

merged 1 commit into from Feb 19, 2013

Conversation

sferik
Copy link
Contributor

@sferik sferik commented Feb 19, 2013

For the purposes of creating diffs, Git prefers files without trailing whitespace. This whitespace is typically inserted by editors such as TextMate. Here are two techniques for TextMate users to avoid inserting extraneous whitespace:

mattwynne added a commit that referenced this pull request Feb 19, 2013
Cleanup trailing whitespace
@mattwynne mattwynne merged commit 6558b80 into cucumber:master Feb 19, 2013
@mattwynne
Copy link
Member

Woot!

Thanks, that's a chore I've been meaning to do for ages.

@mattwynne
Copy link
Member

For bonus points, add something to fail the build if any of it comes back ;)

@sferik sferik deleted the whitespace branch February 19, 2013 18:08
@sferik
Copy link
Contributor Author

sferik commented Feb 19, 2013

@mattwynne I'm not sure how to write a test to prevent it from coming back, but I have written a one-liner that makes it easy to clean up: https://github.com/sferik/dotfiles/blob/9470802cb55e56179a7526cb934c38a11f1af9aa/aliases.sh#L64

@lock
Copy link

lock bot commented Oct 25, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants