Skip to content

clang-tidy: run tool only on source files participating in targets #14736

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

Merged
merged 1 commit into from
Jul 21, 2025

Conversation

Sahnvour
Copy link
Contributor

clang-tidy can't be ran as is on every source file and header

closes #7662

@Sahnvour Sahnvour requested a review from jpakkane as a code owner June 22, 2025 09:59
@bonzini bonzini added this to the 1.9 milestone Jun 22, 2025
@dcbaker
Copy link
Member

dcbaker commented Jul 18, 2025

Code looks good. This should have release snippet and if we have any documentation on the clang-tidy target elsewhere, that needs an update.

@Sahnvour
Copy link
Contributor Author

I'm not aware of documentation on this feature, only saw it in the release notes.
I don't know how release snippets work, are you expecting something from me?

@dcbaker
Copy link
Member

dcbaker commented Jul 19, 2025

A release snippet is just a small bit off markdown that goes in docs/markdown/snippets. When the release is made all of those snippets get gathered up into the "what changed" page. I'd like if you could add something there with this to alert people of this change (it may be surprising to some people).

I also don't seen any generic documentation on the clang-tidy target, so no need to worry about that.

clang-tidy can't be ran as is on every source file and header
@Sahnvour
Copy link
Contributor Author

Updated with snippet.

@dcbaker dcbaker merged commit 6b217e6 into mesonbuild:master Jul 21, 2025
33 checks passed
@dcbaker
Copy link
Member

dcbaker commented Jul 21, 2025

Thanks!

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

Successfully merging this pull request may close these issues.

ninja clang-tidy runs even on source files that are not built by any Meson target
3 participants