This is an archive of the discontinued Mercurial Phabricator instance.

mergeresult: implement a len() function and use it
ClosedPublic

Authored by pulkit on Aug 5 2020, 4:52 AM.

Details

Summary

In next patch we will start yielding from getactions() instead of building and
returning a list. Hence we can no longer rely on that for getting us a length.

Diff Detail

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

Event Timeline

pulkit created this revision.Aug 5 2020, 4:52 AM
pulkit updated this revision to Diff 22302.Aug 6 2020, 7:14 AM
indygreg accepted this revision.Aug 8 2020, 12:23 PM
indygreg added a subscriber: indygreg.
indygreg added inline comments.
mercurial/merge.py
1423

I should read your full series before leaving comments that are addressed in the next patch :)

This revision is now accepted and ready to land.Aug 8 2020, 12:23 PM
This revision was automatically updated to reflect the committed changes.