Closed SyrianSpock closed 7 years ago
I might have missed it, but have you included uncrustify.cfg
in your PR?
@hrilabplymouth No, it was already there. I just ran it on the existing source code and documented usage for future developers.
Thanks, that was long overdue, and the README/precommit hook is a really nice touch :) I'll have a look at whether uncrustify wasn't too zealous before merging, I suppose there is no emergency on your side ?
@qbonnard Take your time, there's no emergency.
Well I did take my time to just rebase and push ;) Many thanks again :)
This PR simply applies the uncrustify config to the source files. The use of uncrustify and integration in pre-commit hook is also documented.