Skip to content

Adjust warnings flags#4801

Merged
nlohmann merged 6 commits intodevelopfrom
fix-warnings
May 31, 2025
Merged

Adjust warnings flags#4801
nlohmann merged 6 commits intodevelopfrom
fix-warnings

Conversation

@nlohmann
Copy link
Owner

@nlohmann nlohmann commented May 30, 2025

A recent Clang update introduced a new Clang Tidy flag portability-avoid-pragma-once and found a Wnrvo warning in doctest. This PR suppresses/fixes these.

In a follow-up ticket, we may want to fix portability-avoid-pragma-once.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@nlohmann nlohmann added this to the Release 3.12.1 milestone May 30, 2025
@nlohmann nlohmann mentioned this pull request May 30, 2025
5 tasks
@nlohmann nlohmann mentioned this pull request May 30, 2025
4 tasks
@coveralls
Copy link

coveralls commented May 30, 2025

Coverage Status

coverage: 99.19%. remained the same
when pulling 0b4cb4c on fix-warnings
into 68c25ae on develop.

Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@github-actions github-actions bot removed the CMake label May 30, 2025
Signed-off-by: Niels Lohmann <mail@nlohmann.me>
@github-actions github-actions bot added the tests label May 30, 2025
nlohmann added 3 commits May 30, 2025 20:03
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 merged commit 82f4f70 into develop May 31, 2025
303 of 317 checks passed
@nlohmann nlohmann deleted the fix-warnings branch May 31, 2025 15:48
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.

2 participants