Replies: 6 comments 1 reply
|
I think this is a great idea and would support it 👍 |
0 replies
|
Any other opinions? @dinkelbachjan @martinmoraga @leonardocarreras? I think that there are a few larger PRs currently open? We should at least wait until those are merged? |
0 replies
|
I support your idea too! |
0 replies
|
Hi @dinkelbachjan, It would nice to hear your opinion on this as well before I start preparing a PR :) |
0 replies
|
Hi @stv0g ! I think this is a nice improvement... Do you think of it like as a pre-commit hook and progressively improve or apply it to the existing codebase in one go? |
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Hi @dinkelbachjan @m-mirz @martinmoraga @leonardocarreras,
We've recently started to use clang-format to automatically format the VILLASnode C++ code.
I would love to introduce clang-format also to DPsim. It seems we have currently various different styles.
Clang-format could harmonize all of that by running a single command.
What are your opinions on this? If the the response is positive, I could prepare an MR implementing this change.
All reactions