I pushed a branch that had an extra blank line. So s_whitespace removed it and pull-request-tester failed due to the modified file. While looking at it, I noticed that s_all had two other failures, one from s_define and s_longlines, both from files I did not touch.
Basically s_all is not failing Jenkins for those kinds of things and so if we don't notice them and the pull-request-tester passes we merge them into develop.
For this particular case, it is shown in the original log:
It was from this PR:
My minor fixes today are:
It doesn't look like s_all:errchk results in a non-zero exit status.