Introduce and use a consistent style

sqfmi / Watchy

Watchy - An Open Source E-Ink Smartwatch

http://www.sqfmi.com

MIT License

2k stars 334 forks source link

Introduce and use a consistent style #161

Closed aneeshdurg closed 2 years ago

aneeshdurg commented 2 years ago

Having a consistent style would make contributing easier for first-time devs! Ideally, this should be run as part of github-actions, but I just wanted to put up a simple PR just introducing the formatting changes to get a discussion around this going.

For auto-formatting, I'm using clang-format, mostly going with llvm style, but with modifications to preserve some of the existing aesthetic decisions around macro formatting (and a possibly controversial choice around assignment formatting).

pyrito commented 2 years ago

I second this. I think this is a great cleanup PR!

sqfmi commented 2 years ago

This is great, thanks!

sqfmi commented 2 years ago

Oops, sorry looks like this commit changed the order of includes in src/Watchy.h and broke the compilation, will have to change that back