This is an archive of the discontinued Mercurial Phabricator instance.

conflictinfo: add more information about the conflict-causing command
AbandonedPublic

Authored by phillco on Sep 14 2017, 2:10 PM.
Tags
None
Subscribers

Details

Summary

In the old version of this extension it was too difficult to determine how to
resume or abort the command that generated conflicts. Let's add that
information so consumers can programatically continue or abort.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

phillco created this revision.Sep 14 2017, 2:10 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 14 2017, 2:10 PM
ryanmce requested changes to this revision.Sep 14 2017, 2:19 PM
ryanmce added a subscriber: ryanmce.

Tests

This revision now requires changes to proceed.Sep 14 2017, 2:19 PM
phillco updated this revision to Diff 1824.Sep 14 2017, 2:31 PM

Should've scrolled up, thanks!

phillco planned changes to this revision.Sep 14 2017, 2:33 PM
phillco updated this revision to Diff 1825.Sep 14 2017, 2:33 PM
phillco added inline comments.Sep 14 2017, 2:34 PM
tests/test-merge-conflictinfo.t
413

I'll see if Aadi was relying on this.

durham accepted this revision.Sep 14 2017, 8:23 PM
durham added a subscriber: durham.

Accepting to unblock, but I think we need to be careful about the possibility of data loss.

hgext3rd/conflictinfo.py
60

This one and updatestate's abort are a bit concerning because it can result in data loss. Is Nuclide actually running these commands for users? Or just telling users? If it's running the commands, maybe we should leave these 2 to_abort's out. Or have some flag that indicates these are not safe to run automatically.

phillco added inline comments.Sep 21 2017, 1:02 PM
hgext3rd/conflictinfo.py
60

It's to present buttons that let the user abort or continue after resolving conflicts, so I think in all cases there'll be data loss on abort. Not sure if update or merge is worse in that case.

ryanmce requested changes to this revision.Oct 13 2017, 9:58 AM

This needs to be rebased; requesting changes until that's done.

This revision now requires changes to proceed.Oct 13 2017, 9:58 AM
phillco updated this revision to Diff 3123.Oct 26 2017, 12:58 AM

Rebased, and I made command = null if there is no conflict, just in case someone was reading it.

phillco updated this revision to Diff 3124.Oct 26 2017, 1:02 AM
phillco updated this revision to Diff 3125.Oct 26 2017, 1:20 AM
phillco abandoned this revision.Oct 30 2017, 11:54 PM

Landed

phillco reclaimed this revision.Oct 30 2017, 11:54 PM

meant to close not abandon