This is an archive of the discontinued Mercurial Phabricator instance.

rebase: log why we didn't use IMM
AcceptedPublic

Authored by phillco on Dec 28 2017, 5:04 PM.
Tags
None
Subscribers
None

Details

Reviewers
quark
Summary

In preparation for adding more conditions that will disable IMM, let's make
this framework more extensible and log the reason IMM wasn't used.

The whynotimm variable name was inspired by the why_not_eden name in
hg/eden/__init__.py.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

phillco created this revision.Dec 28 2017, 5:04 PM
phillco edited the summary of this revision. (Show Details)Dec 28 2017, 5:38 PM
quark accepted this revision.Dec 28 2017, 5:39 PM
quark added inline comments.
hgext/rebase.py
798

+3 spaces

996

same here

This revision is now accepted and ready to land.Dec 28 2017, 5:39 PM