Skip to content

Suppress warnings in NLOHMANN_JSON_SERIALIZE_ENUM#4497

Merged
nlohmann merged 1 commit intodevelopfrom
suppress-clang-tidy-warning-in-macro
Nov 17, 2024
Merged

Suppress warnings in NLOHMANN_JSON_SERIALIZE_ENUM#4497
nlohmann merged 1 commit intodevelopfrom
suppress-clang-tidy-warning-in-macro

Conversation

@nlohmann
Copy link
Owner

Fixes #3924.

  • modernize-type-traits: we use C++11 and don't need to implement our own version of std::is_enum_v here
  • modernize-avoid-c-arrays: we don't want to rely on client code to include <array> when using NLOHMANN_JSON_SERIALIZE_ENUM

@github-actions github-actions bot added the S label Nov 17, 2024
@nlohmann nlohmann added this to the Release 3.11.4 milestone Nov 17, 2024
@nlohmann nlohmann self-assigned this Nov 17, 2024
@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same
when pulling d63ade1 on suppress-clang-tidy-warning-in-macro
into 1c5923e on develop.

@nlohmann nlohmann merged commit 64f68dc into develop Nov 17, 2024
@nlohmann nlohmann deleted the suppress-clang-tidy-warning-in-macro branch November 17, 2024 13:29
slowriot pushed a commit to slowriot/json that referenced this pull request Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

modernize-avoid-c-arrays clang-tidy warning when using NLOHMANN_JSON_SERIALIZE_ENUM macro

2 participants