Hi! I cleaned up your code for you! #8

Merged
merged 1 commit into from Oct 28, 2011

2 participants

@GunioRobot

Hi there!

This is WhitespaceBot. I'm an open-source robot that removes trailing white space in your code, and gives you a gitignore file if you didn't have one!

Why whitespace? Whitespace is an eyesore for developers who use text editors with dark themes. It's not a huge deal, but it's a bit annoying if you use Vim in a terminal. Really, I'm just a proof of concept - GitHub's V3 API allows robots to automatically improve open source projects, and that's really cool. Hopefully, somebody, maybe you!, will fork me and make me even more useful. My owner is funding a bounty to anybody who can add security fixing features to me.

I've only cleaned your most popular project, and I've added you to a list of users not to contact again, so you won't get any more pull requests from me unless you ask. If I'm misbehaving, please email my owner and tell him to turn me off! If this is pull request is of no use to you, please just ignore it.

Thanks!
WhiteSpacebot from Gun.io.

@schwern
Collaborator

@GnuioRobot I'm happy to take the pull request, fortunately Github makes it easy to merge trivial patches, and I appreciate the effort... but... guh.

I'm not going to ask if people care about trailing whitespace, I already know they do WAY TOO MUCH. Rather than going around to every project on Github and patching their source over and over again... how about teaching folks how to configure their editors so they don't show trailing whitespace? Then the whole "eyesore" argument goes away without having to patch the universe continually.

I'm probably talking to a robot.

@schwern schwern merged commit 10bf084 into scrottie:master Oct 28, 2011
@samv

What next? Smoking commits to CPAN/pip/rubygem modules and submitting comments on failed comments? Automatic Perl::Critic, pep8, Perltidy etc?

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