[PATCH] D58731: [clang-tidy] added cppcoreguidelines-explicit-virtual-functions
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 28 04:43:58 PST 2019
JonasToth added a comment.
In D58731#1413498 <https://reviews.llvm.org/D58731#1413498>, @MyDeveloperDay wrote:
> In D58731#1413476 <https://reviews.llvm.org/D58731#1413476>, @lewmpk wrote:
>
> > I'm happy to land this ASAP but I don't have commit rights
>
>
> So one of us could land it for you.. (I've not personally done that before as I'm a bit green too! but I do have commit rights)
>
> Its pretty easy to get commit rights, and given your looking at multiple issues I'd recommend it.. (https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access)
@MyDeveloperDay if you want to try the procedure out, go ahead ;) Otherwise I can land this patch now.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58731/new/
https://reviews.llvm.org/D58731
More information about the cfe-commits
mailing list