Summary
Update the clang-tidy configuration in yscope-dev-utils to treat misc-include-cleaner as a warning instead of an error. This change will allow us to remove NOLINTNEXTLINE(misc-include-cleaner) comments throughout the codebase.
Context
Currently, the codebase contains multiple // NOLINTNEXTLINE(misc-include-cleaner) comments to suppress include-cleaner errors. Since misc-include-cleaner has been changed to generate warnings instead of errors, these suppression comments can be removed.
Tasks
References
Requested by: @sitaowang1998
Summary
Update the clang-tidy configuration in
yscope-dev-utilsto treatmisc-include-cleaneras a warning instead of an error. This change will allow us to removeNOLINTNEXTLINE(misc-include-cleaner)comments throughout the codebase.Context
Currently, the codebase contains multiple
// NOLINTNEXTLINE(misc-include-cleaner)comments to suppress include-cleaner errors. Sincemisc-include-cleanerhas been changed to generate warnings instead of errors, these suppression comments can be removed.Tasks
misc-include-cleaneras warningNOLINTNEXTLINE(misc-include-cleaner)comments from the codebaseReferences
Requested by: @sitaowang1998