[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