Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Use clang-tidy for CI purposes #162

Open
GeorgyKirichenko opened this issue Apr 9, 2024 · 2 comments
Open

Use clang-tidy for CI purposes #162

GeorgyKirichenko opened this issue Apr 9, 2024 · 2 comments

Comments

@GeorgyKirichenko
Copy link
Collaborator

We should choose one of C++ coding guidelines and then enable clang-tidy checker for the project
@TheRandomCharacter
Copy link
Collaborator

We should also make all the changes to .clang-format in the same PR as for clang-tidy, as both will require massive styling related changes to code base when enforced.

@TheRandomCharacter
Copy link
Collaborator

  • AlignAfterOpenBracket: BlockIndent IMHO is better then Align
  • We don't have a ColumnLimit set, this confuses other options, AlignAfterOpenBracket is one of them

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@GeorgyKirichenko @TheRandomCharacter