[riot-notifications] [RIOT-OS/RIOT] Fix common issues with shell scripts (#8212)

Koen Zandberg notifications at github.com
Wed Dec 6 11:34:18 CET 2017


shellcheck is an application to statically check shell scripts for common errors. Most of these consists of small issues with the shell scripts where the problem might only appear in certain edge cases.

As an example, one of the issues is that shellcheck warns for is not having quotes around `${VAR}`. This issue leads in the headerguards check to vague errors as soon as there are spaces in `$PWD`.

I think this could also be a useful addition to the CI to check the shell scripts for common mistakes.

[1]: https://github.com/koalaman/shellcheck

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/issues/8212
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20171206/166daf2d/attachment.html>


More information about the notifications mailing list