[PATCH] D112356: [NFC] Tidy up spelling, grammar, and inconsistencies in clang-tidy documentation

Salman Javed via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 23 04:34:20 PDT 2021


salman-javed-nz added a comment.

In D112356#3082084 <https://reviews.llvm.org/D112356#3082084>, @carlosgalvezp wrote:

> Awesome @salman-javed-nz , thanks for fixing the docs! May I ask how did you create these "smaller commits"? It's very nice to click on each of them and see only what changed. I believe I followed your same procedure (use the "Update Revision" thingy) but it always displays the full commit with all changes, so it's hard to see what each new update did.

I created a patch for each my commits on my local disk like so:

  git show HEAD -U999999 > HEAD.patch
  git show HEAD~1 -U999999 > HEAD~1.patch
  git show HEAD~2 -U999999 > HEAD~2.patch
    (and so on...)

I uploaded them one by one through Phab's web interface. This process is just something I stumbled though myself that worked for me.

I wouldn't be surprised if arcanist has something that makes this easier, but I don't use it so I wouldn't know.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112356



More information about the cfe-commits mailing list