Make dolt log work with ^ ancestor spec
#4594
Conversation
tbantle22
force-pushed
the
taylor/log-ancestor-spec
branch
from
55b3627
to
409dd36
Compare
| run dolt log main^..branch1 | ||
| [ $status -eq 0 ] | ||
| [[ ! "$output" =~ "MAIN" ]] || false | ||
| [[ ! "$output" =~ "AFTER" ]] || false |
There was a problem hiding this comment.
I was expecting AFTER would be in the results... main looks like it's currently pointing at the commit with the AFTER commit message, so main^ should move the exclusion commit to the parent of main, and cause main's current commit to be included in the log. I might just be misreading something though...
There was a problem hiding this comment.
So if the commit graph looks like this:
A - B - F (main)
\
C - D - E (branch1)
dolt log main..branch is essentially dolt log F..E (all branch1, excluding commits from main - E, D, C)
And then dolt log main^..branch is dolt log B..E (all branch1, excluding commits reachable from B, which is still E, D, C because branch1 does not reach F)
There was a problem hiding this comment.
ahhh... yes! Thank you for drawing it out and double checking that. Figured I must just be missing something if everything else was working. Looks great!
No description provided.