Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

ci: Whitespace checker broken

See original GitHub issue

.ci/run.sh whitespace checking function is broken. It should exit non-zero if there is any trailing whitespace in the repo

Issue Analytics

State:
Created 4 years ago
Comments:7 (3 by maintainers)

Top GitHub Comments

1reaction

drigiocommented, Oct 26, 2019

Hey @pdxjohnny initial looks at the code in .ci/run.sh looks good. Also I took a look at previous builds on travis and it has a good history. Running the code locally on pc (I ran only the run_whitespace() function) works properly. When I add an extra whitespace line in any file it duly shows that there are trailing whitespaces.

Checking the exit code by using echo $? returns 1 i.e. non-zero as well. What you think @beinganonymous ? Are you getting some different output @pdxjohnny ? Because I guess Linux distribution and its accompanying toolset (find, grep) etc may matter too.

I am running a distribution that inherits Ubuntu 18.04 LTS.

0reactions

pdxjohnnycommented, Nov 23, 2019

GitHub Actions does not experiance this issue: 70ce150

Top Results From Across the Web

Make Git automatically remove trailing white space before ...

I got frustrated with similar whitespace errors and partial solutions, and wrote a flexible and fairly feature-complete utility which can fix or simply ......

CI environment broken - Devel

Hi, it looks like the CI is broken. All recent Gerrit patches fail currently for strange reasons. check-patch.linters.fc30.x86_64 fails because it cannot ...

Is HTML minification broken? - support

The most that either method will do today is remove blank lines, but it will not strip out all whitespace in the generated...

DatabaseMysqlBase: Remove broken check for Percona Server ...

DatabaseMysqlBase: Remove broken check for Percona Server Percona Server does not include its name in its version suffix. It would be possible to...

white-space - CSS: Cascading Style Sheets - MDN Web Docs

Lines are broken as necessary to fill line boxes. nowrap. Collapses white space as for normal , but suppresses line breaks (text wrapping) ......