mate-desktop / mate-settings-daemon

MATE settings daemon
https://mate-desktop.org
GNU General Public License v2.0
44 stars 47 forks source link

Remove trailing whitespaces #280

Closed tamplan closed 5 years ago

tamplan commented 5 years ago

find . -regextype posix-extended -regex '.*\.(c|h|ac)' -exec sed --in-place 's/[[:space:]]\+$//' {} \+

like rbuj do on https://github.com/mate-desktop/mate-control-center/pull/463.

lukefromdc commented 5 years ago

I don't see any Travis test on this, presumably because they are not on our main repos. Travis autobuilds would presumably be test enough to show that a removal of trailing whitespace didn't alter anything else, as it would be statistically likely that removing or adding a random character would cause a build failiure on at least one and usually all distros

Given the sheer number of these (needed no doubt because this has built up over time), those tests could save all of us a great deal of time

raveit65 commented 5 years ago

Travis-CI builds are there https://travis-ci.org/mate-desktop/mate-settings-daemon/builds/552122285?utm_source=github_status&utm_medium=notification I think updating of github page with with results from Travis-CI (server at Berlin, Germany) is sometimes very slow. In general i notice a delay with connecting some American pages from Europe since 2 years.

sc0w commented 5 years ago

@lukefromdc

I don't see any Travis test on this

You can see it here:

2019-06-30_12-42_1

2019-06-30_12-43

or here:

2019-06-30_12-43_1

2019-06-30_12-44

lukefromdc commented 5 years ago

OK, I see that. If all these do is remove trailing whitespace these should be safe to merge after seeing the test results