- Define clang-format style and enable formatting check in CI
- Fix formatting of Markdown files
- Remove old note about gcc5
- Apply clang-format to source files
Closes T111
Differential D103 Authored by tlatham on Nov 6 2023, 5:47 PM.
Details
Summary
Closes T111 Test Plan Purely formatting changes
Diff Detail
Event Timelinetlatham created this revision. Harbormaster completed remote builds in B271: Diff 428.Nov 6 2023, 5:47 PM2023-11-06 17:47:48 (UTC+0) Comment Actions Since this is purely formatting changes, I think the main thing to review is that the formatting style looks reasonable and that the CI check looks ok. Up to you if you want to look through all the files!! Comment Actions In parallel I need to check what happens if I apply the same formatting in the timedep branch and then try and merge the branches! If that doesn’t go so well we might have to postpone this change untill after merging timedep into master. Comment Actions It looks like it is going to be a nightmare to merge timedep-branch into master if we apply this formatting first (even after applying the same formatting there first). If in the meantime you want to make comments on the clang-format config and CI check, please go ahead. I can still take them into account as part of the above. Comment Actions Clang formatting and CI check looks OK, so proceed with organising the appropriate branches. Harbormaster completed remote builds in B276: Diff 437.Nov 21 2023, 10:47 PM2023-11-21 22:47:58 (UTC+0) This revision is now accepted and ready to land.Nov 24 2023, 4:32 PM2023-11-24 16:32:41 (UTC+0) Comment Actions
Harbormaster completed remote builds in B277: Diff 439.Dec 7 2023, 11:29 AM2023-12-07 11:29:04 (UTC+0) Comment Actions Hi @jback, just a quick note to explain the new changes. Since we are using the el9 LCG_104 platform for the format check, we're running clang 16, so I've updated the clang-format file to account for some new options that have become available since clang 12, which was the version I was originally working with (as per current EvtGen formatting), I also decided to move to 4-space indentation instead of 8-space. This had the side effect that the Doxygen comment blocks looking ragged because clang-format doesn't touch the comment blocks, so I've written a python script to fix those and have added the running of that into the CI check. I also made a few minor adjustments to improve the look of things a bit. I hope this is final now! If you could take a quick look that would be much appreciated. I'd like to release v3r6 next week, if possible. Many thanks!
Revision Contents
Diff 439 |