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

Remove trailing whitespace #16

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@paultcochrane

paultcochrane commented Oct 13, 2015

Some people don't like this in their source code and I thought maybe this PR could be of use. If you consider the change to be superfluous, feel free to simply close the PR unmerged.

@tobyink

This comment has been minimized.

Owner

tobyink commented Jan 31, 2017

Generally speaking if I have two lines of source code with a blank line between them, I tend to indent the blank line the same amount the non-blank lines are indented. My preferred text editor does that by default and I've seen no reason to change it.

@tobyink tobyink closed this Jan 31, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment