[PATCH] D131500: [Docs][CodeReview] Add a small paragraph on adding tokens, NFC.

Whisperity via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 01:43:46 PDT 2022


whisperity accepted this revision.
whisperity added a comment.
This revision is now accepted and ready to land.

Hey!

Sorry, I'm not sure how I missed it. Been on vacations and such... either way, I think documenting this is a good practice.

Do you have commit access or shall I commit on your behalf? If so, what do you want to have as the author name and e-mail for the commit?



================
Comment at: llvm/docs/CodeReview.rst:151
+If you approve of the idea/concept of a patch but feel unqualified to approve,
+another option (other than accepting the patch) is to simply award a token
+(right-hand side in Phabricator) to indicate support without indicating to
----------------
Let's use the actual text verbatim from the GUI? 🙂 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131500/new/

https://reviews.llvm.org/D131500



More information about the llvm-commits mailing list