Add clang-format #110
Add clang-format #110
Conversation
|
Please could you submit these changes as two separate PRs? The .gitignore update looks good and I'll commit it right away, but I want to try out the clang-format file first before deciding whether to add it (I've found that clang-format destroys my hand-crafted nice formatting as often as it fixes things...) |
|
@tcbrindle Seems reasonable! Done in #111 |
...and merged! |
This clang format file was generated using LLVMs tool to "detect" code style from existing code files. It's pretty close to what was already there.
DeveloperPaul123
force-pushed
the
feature/project-files-update
branch
from
89707f0
to
c3d4a15
Compare
DeveloperPaul123
changed the title
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #110 +/- ##
=======================================
Coverage 97.67% 97.67%
=======================================
Files 66 66
Lines 2236 2236
=======================================
Hits 2184 2184
Misses 52 52 ☔ View full report in Codecov by Sentry. |
|
@tcbrindle Any comments on the clang-format? Or any formatting changes you'd like to see in the config file? |
|
I do like the idea of having a project-wide clang format config to keep everything consistent, and to make life easier for new contributors. The only problem is that I'm very fussy when it comes to how I like my code formatted! I still need to have a play with all the clang-format options until I find the combination I like. |
|
I completely understand. If you have examples of styling that you want to keep that |
This clang-format was created using LLVM's tool to "detect" the format from existing code. From my testing, it's quite close.