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

Purge trailing whitespace in source files #8

Merged
merged 3 commits into from Mar 14, 2018

Conversation

@paultcochrane
Copy link
Contributor

@paultcochrane paultcochrane commented Mar 14, 2018

Trailing whitespace is seen in some projects as bad practice (e.g. the Linux kernel) and is hence explicitly forbidden; other projects see its removal as plain nit-picking. This PR is submitted in the hope that it is helpful, however if you don't see any need to remove such whitespace I'm happy if you close the PR as unmerged. I've split the commits so that you can cherry pick them as required. If you want these changes to be bundled into one commit, just let me know and I'll update the PR and resubmit.

@karpet karpet merged commit 620e766 into karpet:master Mar 14, 2018
1 check passed
@paultcochrane paultcochrane deleted the pr/purge-trailing-whitespace branch Mar 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants