[clang-tools-extra] [clang-tidy] Add modernize-nlohmann-json-explicit-conversions check (PR #126425)
Congcong Cai via cfe-commits
cfe-commits at lists.llvm.org
Sun Feb 9 18:07:55 PST 2025
HerrCai0907 wrote:
I am not sure whether we should accept this kind of "common" used library's custom check in clang-tidy.
- powerful and easy used tools which can help lots of C++ dev
- longer CI time
- larger binary size
I think we should a guideline this kind of cases to avoid to waste contributors effort. It is frustrating to write code and find out it can't possibly be merged.
@PiotrZSL @5chmidti @carlosgalvezp @AaronBallman WDYT?
https://github.com/llvm/llvm-project/pull/126425
More information about the cfe-commits
mailing list