Skip to content

Conversation

maribu
Copy link
Member

@maribu maribu commented Apr 23, 2025

Contribution description

This adds a default .clang-tidy configuration for additional linitng pleasure when using an editor with LSP support and clangd as language server.

Testing procedure

A LSP enabled editor that uses clangd should now provide extra liniting pleasure, after make compile-commands -C path/to/app is used to create a compile_commands.json.

Issues/PRs references

None

This adds a default .clang-tidy configuration for additional linitng
pleasure when using an editor with LSP support and clangd as language
server.
@maribu maribu added Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: tools Area: Supplementary tools labels Apr 23, 2025
@github-actions github-actions bot removed the Area: tools Area: Supplementary tools label Apr 23, 2025
@riot-ci
Copy link

riot-ci commented Apr 23, 2025

Murdock results

✔️ PASSED

34c0964 tools: add default .clang-tidy configuration

Success Failures Total Runtime
10299 0 10299 10m:10s

Artifacts

@crasbe
Copy link
Contributor

crasbe commented Apr 24, 2025

So this essentially reverts #16791. I just checked why it was added back then and since it was you who added it, it's probably fine to remove it now that an official .clang-tidy configuration will be added.

@maribu maribu added this pull request to the merge queue Apr 24, 2025
Merged via the queue into RIOT-OS:master with commit 00a2791 Apr 24, 2025
29 checks passed
@maribu
Copy link
Member Author

maribu commented Apr 24, 2025

Thx :-)

@maribu maribu deleted the dist/clang-tidy branch April 24, 2025 12:37
@Teufelchen1 Teufelchen1 added this to the Release 2025.07 milestone Jul 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants