[clang-tools-extra] [clang-tidy] Add modernize-nlohmann-json-explicit-conversions check (PR #126425)
Carlos Galvez via cfe-commits
cfe-commits at lists.llvm.org
Sun Feb 9 23:15:26 PST 2025
carlosgalvezp wrote:
I have some mixed feelings as well, this could open the door for clang-tidy becoming a tool for modernizing "any" 3rd-party library usage. It feels like instead nlohmann/json should be the one responsible for providing a "migration path" / tooling for these types of deprecated features.
It could also easily become a mess when we have to deal with / support different versions of the libraries.
Thus I would vote for keeping clang-tidy as a general tool for C++/STL only. Perhaps these types of utilities could be implemented as a clang-tidy **plugin** in the respective 3rd-party repository?
https://github.com/llvm/llvm-project/pull/126425
More information about the cfe-commits
mailing list