Skip to content
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

Move to a newer clang format #848

Closed
kuznetsss opened this issue Sep 13, 2023 · 5 comments
Closed

Move to a newer clang format #848

kuznetsss opened this issue Sep 13, 2023 · 5 comments
Assignees
Labels
ci CI related issues enhancement New feature or request

Comments

@kuznetsss
Copy link
Collaborator

We are using quite outdated clang-format-11. E.g. clang-format 14 allows to force east const https://clang.llvm.org/docs/ClangFormatStyleOptions.html#qualifieralignment

@godexsoft godexsoft added enhancement New feature or request ci CI related issues labels Sep 13, 2023
@godexsoft
Copy link
Collaborator

Hopefully also concepts code can be formatted properly now? and some lambdas in our codebase could benefit from better formatting too if we can now tweak it.

@kuznetsss
Copy link
Collaborator Author

Try to adjust config to have formatting for

if
     <one line code>
ellse
   <one line code>

statements without { }

@kuznetsss
Copy link
Collaborator Author

Force east const style

@kuznetsss
Copy link
Collaborator Author

#864 (comment)

@kuznetsss
Copy link
Collaborator Author

@godexsoft godexsoft self-assigned this Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CI related issues enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants