Makefile: return error if a check fails
authorDarshaka Pathirana <dpat@syn-net.org>
Sat, 24 Apr 2021 22:25:29 +0000 (00:25 +0200)
committerDarshaka Pathirana <dpat@syn-net.org>
Fri, 3 Dec 2021 11:54:54 +0000 (12:54 +0100)
commit02a237b8633dca400bd1d08f87c24cea30933f92
tree1cfb084ae0e67739bbf45cb981891288764f75e9
parent98d1b69753568d367f9602c4102e2b984a17d497
Makefile: return error if a check fails

Make target pythoncheck (and codecheck) did not fail if the last tested
file was error-free. Now we set RETURN=1 if only one check fails.

While at it also created separate targets for flake8, black and isort,
but also added --keep-going option when running 'make codecheck' so that all
targets are run.

Added @ in front of each command to hide the command which makes the
output more readable.

Spellcheck SC1117 (Backslash is literal in "\n") is no longer emitted
(with shellcheck after v0.5), thats why it is removed.

Please note, that spellintian does not return an error code if an
incorrect spelling is found, that is why this ugly hack had to be
implemented.
.github/workflows/tests.yml
Makefile