Page MenuHomePhabricator

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

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



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

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

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

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.