[Differential] [Accepted] D62: rebase: add config to move rebase into a single transaction

2017-07-12 Thread phillco (Phil Cohen)
phillco accepted this revision as: phillco. phillco added a comment. This revision is now accepted and ready to land. Actual LGTM. Though, it's interesting, it looks like I have the ability to accept as `hg`. I presume I'll need to keep unchecking that so I don't accidentally pretend to

[Differential] [Accepted] D62: rebase: add config to move rebase into a single transaction

2017-07-12 Thread martinvonz (Martin von Zweigbergk)
martinvonz accepted this revision. martinvonz added inline comments. INLINE COMMENTS > util.py:592 > > +class nullcontextmanager(object): > +"""A no-op context manager. The body looks more like a "nulltransaction" (I think close(), release(), abort() are part of transaction's interface),