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 white spaces #31583

Merged
merged 7 commits into from Mar 2, 2016
Merged

Remove trailing white spaces #31583

merged 7 commits into from Mar 2, 2016

Conversation

vutny
Copy link
Contributor

@vutny vutny commented Mar 1, 2016

What does this PR do?

Cleans the files in the project by removing unnecessary trailing whitespaces.
Inspired by: https://programmers.stackexchange.com/questions/121555/why-is-trailing-whitespace-a-big-deal

@rallytime
Copy link
Contributor

@vutny It looks like this change is causing some test errors. Would you mind taking a look? If you get stuck let us know and we can help out. There are 2 errors that are likely not related, but since the fileserver failures are in both testing VMs, those should be cleaned up:

https://jenkins.saltstack.com/job/salt-pr-rs-ubuntu14.04-n/10263/

@rallytime rallytime added the pending-changes The pull request needs additional changes before it can be merged label Mar 1, 2016
@vutny
Copy link
Contributor Author

vutny commented Mar 2, 2016

@rallytime Hurray, all green!

@rallytime rallytime removed the pending-changes The pull request needs additional changes before it can be merged label Mar 2, 2016
@rallytime
Copy link
Contributor

Thanks!

rallytime pushed a commit that referenced this pull request Mar 2, 2016
@rallytime rallytime merged commit 36ce240 into saltstack:2015.8 Mar 2, 2016
@vutny vutny deleted the remove-trailing-white-spaces branch March 2, 2016 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants