NagyDonat wrote: > > Since this is a CERT check, please also add an alias to that module > > Is it correct to add a CERT alias even if the rule is only partially covered > by the check? For rule DCL56-CPP the > `cppcoreguidelines-interfaces-global-init` would be needed additionally.
I think the most elegant solution would be introducing a new check `cert-dcl56-cpp` that enables both this checker and `cppcoreguidelines-interfaces-global-init`. However, I think it would be better to place this into a separate follow-up commit (although I'm not totally opposed to extending this PR if others prefer that.) Do we have any precedent for this "alias for a combination of checkers" situation? @carlosgalvezp What do you think about this? https://github.com/llvm/llvm-project/pull/175342 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
