Skip to content

Suppress clang-analyzer-webkit.NoUncountedMemberChecker #4701

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 11 commits into from
Mar 29, 2025

Conversation

nlohmann
Copy link
Owner

@nlohmann nlohmann commented Mar 20, 2025

A new Clang-Tidy version introduced a new check clang-analyzer-webkit.NoUncountedMemberChecker which we disable.

Furthermore, check misc-const-correctness had some false positives.

Finally, we added Clang 20 to the list of used compilers since Clang-latest is now Clang 21.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@github-actions github-actions bot added the S label Mar 20, 2025
@coveralls
Copy link

coveralls commented Mar 20, 2025

Coverage Status

coverage: 99.186%. remained the same
when pulling fb2662d on suppress-clang-tidy
into 11a835d on develop.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@github-actions github-actions bot added the tests label Mar 20, 2025
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann added this to the Release 3.11.4 milestone Mar 24, 2025
@nlohmann nlohmann added the review needed It would be great if someone could review the proposed changes. label Mar 24, 2025
@nlohmann nlohmann merged commit b477d2b into develop Mar 29, 2025
133 checks passed
@nlohmann nlohmann deleted the suppress-clang-tidy branch March 29, 2025 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI documentation M review needed It would be great if someone could review the proposed changes. tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants