-
Notifications
You must be signed in to change notification settings - Fork 396
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Align qualifiers using clang-format #2647
Align qualifiers using clang-format #2647
Conversation
@@ -67,6 +67,8 @@ PenaltyBreakTemplateDeclaration: 0 | |||
PenaltyExcessCharacter: 10 | |||
PenaltyIndentedWhitespace: 0 | |||
PenaltyReturnTypeOnItsOwnLine: 10 | |||
QualifierAlignment: Custom # Experimental | |||
QualifierOrder: [inline, static, constexpr, const, volatile, type] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JohanMabille @tdegeus here is the setting for the ordering.
I'm happy to change it to east-const if you'd like ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice, makes the code significantly more readable!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no way ;)
Is |
It is Yaml indeed. Perhaps we can the file explicitly to the yaml pre-commit hooks. |
That would be great indeed! |
This seems to do the trick, but it seems to not be a hook for sorting yaml (which is probably better, sorting is not always a good fit for configs). |
Checklist
Description
This option is experimental, let's see if it works.