Page MenuHomePhabricator

merge: make hg.merge() take a context instead of a node
Needs RevisionPublic

Authored by martinvonz on Jan 24 2020, 7:16 PM.

Details

Reviewers
durin42
Group Reviewers
hg-reviewers
Summary

Many callers already have a repo, so we might as well pass
that. merge.update() will look up the context object later. This
patch is part of making it so we pass around the context object all
the way down instead.

I also removed the repo argument since it can be retrieved from the
context.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

martinvonz created this revision.Jan 24 2020, 7:16 PM
martinvonz updated this revision to Diff 19605.Sat, Jan 25, 1:05 AM
durin42 requested changes to this revision.Wed, Jan 29, 6:09 PM
durin42 added a subscriber: durin42.

Breaks test-merge2.t but otherwise seems like a good series.

This revision now requires changes to proceed.Wed, Jan 29, 6:09 PM

Breaks test-merge2.t but otherwise seems like a good series.

Doesn't break it for me (I think I just re-ran that test on all the commits in this series). I guess it's a (semantic) conflict with some other of my patches that have recently been queue. I'll rebase and try again once you're done with your current batch of queuing.

Breaks test-merge2.t but otherwise seems like a good series.

Doesn't break it for me (I think I just re-ran that test on all the commits in this series). I guess it's a (semantic) conflict with some other of my patches that have recently been queue. I'll rebase and try again once you're done with your current batch of queuing.

Rebase done and it still passes for me. How does it fail for you? I wonder if it's flaky. I've seen one of those tests be flaky before.