( )⚙ D8308 phabricator: record all local commits used to create a Differential revision

This is an archive of the discontinued Mercurial Phabricator instance.

phabricator: record all local commits used to create a Differential revision
ClosedPublic

Authored by mharbison72 on Mar 20 2020, 5:24 PM.

Details

Summary

Arcanist records all of the commits that it squashes into a single review, and
that info will be helpful when adding similar functionality. This info is used
when submitting an updated review, so that the extension can recalculate the old
diff and see if a new one is necessary, or if it is just a property update. It
also shows on the commits tab in the Revision Contents section.

When submitting in the usual 1:1 commit to review mode, the wire protocol is
unchanged.

The content of hg:meta is a bit odd, but such is the problem when folding
several commits. The choice for the parent node is obvious, but the node
value uses the tip commit because that seems more natural, and is used elsewhere
to look up the previous diff when updating. The rest of the attributes follow
from there.

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

mharbison72 created this revision.Mar 20 2020, 5:24 PM
Alphare accepted this revision.Mar 25 2020, 2:18 PM
This revision was not accepted when it landed; it landed in state Needs Review.
This revision was automatically updated to reflect the committed changes.