Skip to content

Add clang-format to pre-commit#693

Open
Spartan322 wants to merge 2 commits into
masterfrom
add/clang-format
Open

Add clang-format to pre-commit#693
Spartan322 wants to merge 2 commits into
masterfrom
add/clang-format

Conversation

@Spartan322

@Spartan322 Spartan322 commented Jun 21, 2026

Copy link
Copy Markdown
Member

Update clang-format IncludeCategories
Remove UseCRLF from clang-format
Change UseTab to ForIndentation in clang-format
Change BreakConstructorInitializers to AfterColon in clang-format

@Spartan322
Spartan322 requested a review from a team as a code owner June 21, 2026 18:16
@Spartan322 Spartan322 added enhancement dependencies Pull requests that update a dependency file topic:codestyle labels Jun 21, 2026
@Spartan322
Spartan322 force-pushed the add/clang-format branch 3 times, most recently from 7bf8c8d to 00706ca Compare June 27, 2026 16:35
@Spartan322
Spartan322 force-pushed the add/clang-format branch 3 times, most recently from 73968dc to 9d0ab7d Compare July 10, 2026 20:10
BrickPi
BrickPi previously approved these changes Jul 19, 2026

@BrickPi BrickPi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, big win for maintainability long-term.

Update clang-format IncludeCategories
Remove UseCRLF from clang-format
Change UseTab to ForIndentation in clang-format
Change BreakConstructorInitializers to AfterColon in clang-format
Change AlwaysBreakAfterReturnType to ExceptShortType
Change PenaltyReturnTypeOnItsOwnLine to 500
Change AllowShortEnumsOnASingleLine to false
Change AlignOperands to AlignAfterOperator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement topic:codestyle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants