( )⚙ D6037 absorb: run cleanupnodes() within transaction also when not using obsmarkers

This is an archive of the discontinued Mercurial Phabricator instance.

absorb: run cleanupnodes() within transaction also when not using obsmarkers
ClosedPublic

Authored by martinvonz on Feb 28 2019, 11:03 AM.

Details

Summary

scmutil.cleanupnodes() schedules stripping to be done after the
current transaction, so we can safely run it within the
transaction. This also means that the phase will be updated within the
transaction, which I believe means that the initial (possibly
incorrect) phase will not visible.

Diff Detail

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

Event Timeline

martinvonz created this revision.Feb 28 2019, 11:03 AM
This revision was automatically updated to reflect the committed changes.