[PATCH] D74990: [arcconfig] Default base to previous revision

Shoaib Meenai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 15:25:18 PST 2020


smeenai created this revision.
smeenai added reviewers: jyknight, phosek, rnk.
Herald added a project: LLVM.

When you run `arc diff`, arc defaults to uploading all the changes you
have against the upstream branch into a single patch. This is almost
never what you want for stacked commits (patch series); you only want to
submit the changes done by the current patch. It's also come up as a
point of confusion in the Phabricator vs. GitHub PRs discussion, for
example. Configure arc to only upload your current patch by default,
which I think is a much more suitable default for LLVM developers.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D74990

Files:
  .arcconfig


Index: .arcconfig
===================================================================
--- .arcconfig
+++ .arcconfig
@@ -1,5 +1,6 @@
 {
   "phabricator.uri" : "https://reviews.llvm.org/",
   "repository.callsign" : "G",
-  "conduit_uri" : "https://reviews.llvm.org/"
+  "conduit_uri" : "https://reviews.llvm.org/",
+  "base": "git:HEAD^"
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74990.246009.patch
Type: text/x-patch
Size: 340 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200221/0e515e5b/attachment.bin>


More information about the llvm-commits mailing list