Looks good. One change discussed on IRC: instead of a variable called diverged_and_not_current_upstream, it may be easier to read to have one called diverged and another called current_upstream.
Thanks for the drive-by cleanups.
« Back to merge proposal
Looks good. One change discussed on IRC: instead of a variable called diverged_ and_not_ current_ upstream, it may be easier to read to have one called diverged and another called current_upstream.
Thanks for the drive-by cleanups.