Reshef Elisha
12/04/2023, 11:18 AMTyler Nieman
12/04/2023, 6:00 PMgt pr --github
would be helpful for when i cannot manage the PR through graphite.Cal Courtney
12/05/2023, 3:30 PMRebase after each merge
and ignore optional CI checks
. But, I notice that when I have Merge when ready
on, I don’t get the same controls. Is there a setting where I could change the MWR behaviour?Reshef Elisha
12/05/2023, 6:48 PMgt restack; gt submit
When merging part of a stack, main<-A<-B<-C<-D
, let's say I merge A
, I'd expect to end up with main<-B<-C<-D
. NOT stale-A<-B<-C<-D
.
This is actually insidious, and possibly a bug, since if I then naively hit "merge" on B, I create a situation that I can't get out of using the UI, since now stale branch A has the commit from B. This is not the behavior on phabricator and is definitely not the user expected behavior. Github PRs auto-update their branch when merge stacked PRs, so this is also not the behavior on github.
I've seen on the slack several times the recommendation that after merging a bottom commit, we should run gt restack; gt submit
, but this just tells me that many people fall for it, and the cleanup is very annoying, involving several git operations I don't want to do day-to-day. I'd definitely consider this a bug if I didn't know it's intended by the Graphite team.Ella Rohm-Ensing
12/05/2023, 11:42 PMAlbert
12/06/2023, 9:11 AMgt log
show whether a PR has checks busy running (or a summary of checks done / outstanding)?Jeff Huang
12/06/2023, 8:56 PMCasper Kizewski
12/07/2023, 10:06 AMLeslie Lam
12/07/2023, 11:10 PMNoé RUBINSTEIN
12/08/2023, 11:02 AMgt modify --edit
the single commit of a PR, then gt submit
it, it should update the title of the pull request (and possibly the body but I understand this can be more finicky)Harrison Katz
12/11/2023, 2:57 PMgt create
default to a selection? I don't like the extra click everytime when I just want to create a blank branch and then add commits later.Tyler Laprade
12/11/2023, 4:18 PMBen Jaffe
12/11/2023, 5:00 PMBen Jaffe
12/11/2023, 9:59 PMNathan Flurry
12/12/2023, 3:51 AMJohn Mikael Gundersen
12/12/2023, 7:48 AMNathan Flurry
12/12/2023, 8:09 AMgt co -
similar to git checkout -
Albert
12/12/2023, 12:36 PMAre you sure you want to abandon splitting?
prompt when you accidentally leave a big split by hunk operation 😭 😭 ?Tyler Laprade
12/12/2023, 3:23 PMBobby Christopher
12/12/2023, 3:38 PMgt modify
in the terminal to see the error. Could we find a way to better surface these issues? Maybe in this lil window.Jordan Nunez
12/12/2023, 6:34 PMgt undo restack
. I want to undo last restack and change my local state to its previos version stack. useful for the infrequent – but not rare – base rebaseTyler Laprade
12/12/2023, 8:32 PMChristopher Katz
12/12/2023, 8:51 PMChristopher Katz
12/12/2023, 9:30 PMgt sync
that it would tell me what I have to doChristopher Katz
12/12/2023, 9:37 PMJordon Wing
12/12/2023, 10:04 PMMatt Demichele
12/13/2023, 5:30 PMElla Rohm-Ensing
12/13/2023, 5:59 PMgt checkout
doesn't pass through to git checkout
. I would love though if it could be used with the --ours
/ --theirs
option for resolving rebases, since restacking often ends up in resolving merge conflicts this wayJeff Huang
12/13/2023, 7:01 PMMartin Ek
12/14/2023, 4:47 AMMerge when ready
with the merge queue it waits with rebasing a pull request until the build has succeeded. This seems like a reasonable conservative choice but it would be nice to be able to tell it to rebase as soon as it sees that the PR is out of date with main so that if all the tests are going to pass it will be ready to merge sooner rather than later (i.e., having to wait for only one set of tests to run instead of two)