This is an archive of the discontinued Mercurial Phabricator instance.

infinitepush: move bundle storing logic to its own function
ClosedPublic

Authored by durham on Sep 14 2017, 5:17 PM.
Tags
None
Subscribers
None

Details

Summary

Previously the logic that created a bundle from a changegroup part and put it in
the store was all in the same function. Let's split it up.

This will be useful in a later part where we stop using the default bundle2 part
handling logic and instead handle each part manually (allowing us to store other
parts in the bundle).

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

durham created this revision.Sep 14 2017, 5:17 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 14 2017, 5:17 PM

What's the test plan?

stash accepted this revision.Sep 15 2017, 12:14 PM

Assuming that you ran the unit tests

This revision is now accepted and ready to land.Sep 15 2017, 12:14 PM

Yep, test plan on all of this stack is ran the tests

This revision was automatically updated to reflect the committed changes.