Page MenuHomePhabricator

Add relations between commits
Needs RevisionPublic

Authored by sascha-egerer on Oct 26 2013, 5:29 PM.
Tags
None
Referenced Files
F14020587: D7420.diff
Wed, Nov 6, 12:53 AM
F13996020: D7420.diff
Wed, Oct 23, 5:27 PM
Unknown Object (File)
Sep 23 2024, 9:31 AM
Unknown Object (File)
Sep 23 2024, 9:28 AM
Unknown Object (File)
Sep 23 2024, 9:28 AM
Unknown Object (File)
Sep 6 2024, 9:20 AM
Unknown Object (File)
Aug 15 2024, 3:07 PM
Unknown Object (File)
Aug 11 2024, 2:17 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4026: Add relations between commits
Summary

Allow to add relations between commits.
This is useful if you only use audit reviews and
want to add some information when something is fixed
in antoher commit.

Fixes T4026

Test Plan

Go to a commit and add a relation
to another commit.

Diff Detail

Branch
master
Lint
Lint Warnings
Unit
Tests Passed

Event Timeline

The only two "relationships" I've seen in the wild are "X fixes Y" and "X reverts Y". Do you have more than this?

I'd like to formalize "fixes" and "reverts" as first-class relationships. They're relevant for us in other cases (i.e., Diffusion and other applications like Differential and arc can work better if they know that two commits are related by "X reverts Y" than if they just know that they're related), and we can derive them automatically in some cases.

D5846 adds (and automatically populates) a "reverts" relationship, although it needed a bit more work so I never finished it.

T2542 discusses formalizing the "fixes" relationship.

Are there other relationships here, beyond "reverts" and "fixes"?

Are there other relationships here, beyond "reverts" and "fixes"?

I don't think so. Im fine with your changes.