[WIP] C++ linter and formatter (experimental) #320

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.

Sign up for GitHub

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

Jump to bottom

Open

JanPetterMG wants to merge 26 commits into main from clang-workflow

+1,941 −1,602

Contributor

JanPetterMG commented Jan 30, 2026 •

edited

Loading

Implement clang-tidy as well as some changes to clang-format config. Recommend xaver.clang-format for VS Code and use it as the default formatter.

Key changes

Clang workflow
Clang-tidy config
Use xaver.clang-format as formatter in VS Code

Impact

On code smells, a code review will be submitted with suggested changes.

JanPetterMG added the no-changelog label

JanPetterMG marked this pull request as ready for review

January 31, 2026 12:32

This comment was marked as outdated.

Sign in to view

JanPetterMG marked this pull request as draft

January 31, 2026 12:36

JanPetterMG marked this pull request as ready for review

January 31, 2026 12:41

github-actions bot dismissed their stale review

January 31, 2026 12:45

outdated suggestion

This comment was marked as outdated.

Sign in to view

github-actions bot dismissed their stale review

January 31, 2026 12:49

outdated suggestion

JanPetterMG marked this pull request as draft

January 31, 2026 12:51

JanPetterMG marked this pull request as ready for review

January 31, 2026 12:55

This comment was marked as outdated.

Sign in to view

JanPetterMG marked this pull request as draft

January 31, 2026 12:59

JanPetterMG marked this pull request as ready for review

January 31, 2026 13:08

github-actions bot dismissed their stale review

January 31, 2026 13:09

outdated suggestion

This comment was marked as outdated.

Sign in to view

github-actions bot dismissed their stale review

January 31, 2026 13:09

outdated suggestion

JanPetterMG marked this pull request as draft

January 31, 2026 13:16

JanPetterMG marked this pull request as ready for review

January 31, 2026 13:48

This comment was marked as outdated.

Sign in to view

github-actions bot dismissed their stale review

January 31, 2026 13:55

outdated suggestion

This comment was marked as outdated.

Sign in to view

github-actions bot dismissed their stale review

January 31, 2026 14:21

outdated suggestion

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view


          More clang-tidy exclusions + some formatted code (draft)

0ba9a9b

github-actions bot added the firmware label

JanPetterMG marked this pull request as ready for review

February 1, 2026 14:33

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view


          WIP: Formatting and minor refactors

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

JanPetterMG changed the title ~~C++ linter and formatter~~ [WIP] C++ linter and formatter

WIP

b46d5ef

github-actions bot added the extra label

JanPetterMG changed the title ~~[WIP] C++ linter and formatter~~ [WIP] C++ linter and formatter (experimental)

WIP

546ec51

This comment was marked as outdated.

Sign in to view

WIP

0fb453f

JanPetterMG marked this pull request as draft

February 2, 2026 02:58

JanPetterMG added 11 commits

February 2, 2026 15:36

WIP

6454df6

WIP

3ccc632

WIP

b4a032a

WIP

5bfc700

WIP

994cc11

WIP

c5749d4

WIP

83ca097

WIP

fd2cef8

WIP

1804e60

WIP

462352e

WIP

8d74a7d

JanPetterMG marked this pull request as ready for review

February 3, 2026 20:30

JanPetterMG added 2 commits

February 3, 2026 22:12

WIP

48d216e

WIP

55917fd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore extra firmware