This is an archive of the discontinued Mercurial Phabricator instance.

merge: move initial handling of mergeactions near to later one
ClosedPublic

Authored by pulkit on Sep 5 2020, 3:08 AM.

Details

Summary

We build mergeactions in the beginning and use it in end. Let's build it just
before where it will be used. Helps making code much easier to understand.

Diff Detail

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

Event Timeline

pulkit created this revision.Sep 5 2020, 3:08 AM
marmoute accepted this revision.Sep 11 2020, 8:56 AM
marmoute added a subscriber: marmoute.

This kind of massive code block move in a huge function make me think we should slice it into smaller function. What do you think?

pulkit updated this revision to Diff 22638.Sep 14 2020, 7:13 AM
pulkit updated this revision to Diff 22660.Sep 17 2020, 5:39 AM
indygreg accepted this revision.Sep 17 2020, 10:10 PM
This revision is now accepted and ready to land.Sep 17 2020, 10:10 PM