[PATCH] D79964: Update 'git push' command in GettingStarted guide

[Eric Christopher via Phabricator via llvm-commits]

echristo added inline comments.


================
Comment at: llvm/docs/Phabricator.rst:218
   ninja check-$whatever # Rerun the appropriate tests if needed.
-  git push
+  git push origin arcpatch-D<Revision>:master
 
----------------
mehdi_amini wrote:
> The generic way of pushing would rather be I believe:  `git push origin HEAD:master` what about using this instead? It is closer to ensuring you're pushing what you just tested (we should like tell folks to check that their local state is clean as well)
> 
> (note that origin is read-only locally for me, but I assume that for such doc we can assume a "default" setup)
+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79964