Switch to uncrustify#517
Conversation
jcfr
left a comment
There was a problem hiding this comment.
Thanks for working on this:
-
Considering cherry picking commit titled
chore: Remove obsolete runastyle scriptsfrom #516 -
Add a second commit
chore: Add settings corresponding to uncrustify 0.80.1 Introduce settings based of similar settings copied from danmar/cppcheck@b4f07cdbb. -
Add a third commit:
style: Consistent formatting of C++ sources using uncrustify This commit updates the sources file based on the settings introduced in previous commit. -
Either in the context of this pull request or in a follow-up one, add
.git-blame-ignore-revs -
Add
pre-commit.ymlworkflow with theuncrustifyhook enabled.
Footnotes
Sorry, something went wrong.
4f106fe to
ecf23c1
Compare
August 28, 2025 17:40
ecf23c1 to
18cfb6a
Compare
August 28, 2025 17:44
|
@jcfr thanks for the reviews. there was some mess in my first commit to cleanup as you saw. but I hope it didn't add any extra changes now. |
Sorry, something went wrong.
|
The commit should probably have been split as suggested in #517 (review). Indeed, referencing 18cfb6a in While this is not a critical in the current situation, in the future, I suggest to avoid mixing changes. |
Sorry, something went wrong.
No description provided.