whisperity removed a reviewer: bzcheeseman. whisperity added a comment. This revision now requires review to proceed.
In D131319#3708667 <https://reviews.llvm.org/D131319#3708667>, @bzcheeseman wrote: > This is great, thank you for doing this! I'm not a competent reviewer for the > actual clang-tidy code changes but the +1 for the idea :) The problem with the approval here is that a single approval will turn the patch into a fully approved state, which breaks the dashboards for people added to the patch (i.e., other reviewers will think the patch is already approved, and thus perhaps will not consider putting effort into reviewing it!). However, I think you should try the //Award Token// option from the menu on the right! Somewhere the awarded tokens should show up on the patch, tallying up support! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131319/new/ https://reviews.llvm.org/D131319 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
