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

Remove trailing whitespace #69

merged 3 commits into from Oct 4, 2018


Copy link

@paultcochrane paultcochrane commented Oct 3, 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. BTW: I've split up the PR into several commits to aid checking the diffs and just in case you want to cherry pick the commits. If you want the commits merged into a single commit, just let me know and I'll make the change and resubmit the PR.

@rizen rizen merged commit b1bfc5c into rizen:master Oct 4, 2018
@paultcochrane paultcochrane deleted the paultcochrane:pr/remove-trailing-whitespace branch Oct 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.