[riot-notifications] [RIOT-OS/RIOT] uncrustify: add auto uncrustify with blacklist (#8519)

Kaspar Schleiser notifications at github.com
Fri Jan 25 14:58:06 CET 2019


>     1. Code beautifying
>     2. Code-style checking

Could you recap where you think they differ?

>  The intention of this PR was to add a CI helper, that's why I closed it.

Yes, that's why I re-opened it. The idea is awesome. I want this to be run on every PR. 

>     * Improve Uncrustify support on RIOT and user integration
>     * Use Vera++ in the CI to check coding style.
> 
> In practice, Vera++ would never be run by a developer unless someone wants to run CI checks locally

To me this defeats the purpose of an automatic code formatting tool, if the developer does not run it.

Please let us converge to a point where we do not have to deal with code formatting anymore.

Last time I looked, Vera++ was far from ```apt-get install vera++; vera++ vera-riot.cfg```. IMO, that ease of use is a requirement.

-- 
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/pull/8519#issuecomment-457580597
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190125/b1736fdf/attachment.html>


More information about the notifications mailing list