[PATCH] D64893: Ask confirmation when `git llvm push` will push multiple commits

Yitzhak Mandelbaum via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 10:07:48 PDT 2019


ymandel added inline comments.


================
Comment at: llvm/utils/git-svn/git-llvm:116
+        query = input('%s (y/n): ' % (prompt))
+        if query == '' or not query[0].lower() in ['y','n']:
+           print('Expect y or n!')
----------------
Maybe default to "no" (and indicate that with "(y/N)" in the query) when the user doesn't enter either one?


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

https://reviews.llvm.org/D64893





More information about the llvm-commits mailing list