Page MenuHomePhabricator
Create Task
Maniphest T4026

Add relations between commits
Closed, DuplicatePublic
Actions

Description

Currently it is not possible to link commits/revisions together. This would be helpful if you only use the "audit" mode.
If somebody raises a concern you can than add a relation with a type like "fixed in rFOOO123".
The relation type should also be adjustable for each relation (fixed, done, introduced ...)

We are not using Maniphest and so we cant use this at central relation.

Revisions and Commits

rP Phabricator
Needs RevisionD7420 Add relations between commits

Related Objects

Event Timeline

sascha-egerer created this task.Oct 25 2013, 7:08 PM
sascha-egerer raised the priority of this task from to Needs Triage.
sascha-egerer updated the task description. (Show Details)
sascha-egerer added a project: Phabricator.
sascha-egerer added a subscriber: sascha-egerer.
sascha-egerer updated the task description. (Show Details)Oct 25 2013, 7:11 PM
sascha-egerer edited this Maniphest Task.Oct 26 2013, 5:29 PM
chad edited projects, added Audit, Differential; removed Phabricator.Nov 3 2014, 1:11 AM
joshuaspence added a subscriber: joshuaspence.May 30 2015, 1:43 AM
chad mentioned this in T9533: Auto-create Maniphest tasks for concerns raised through Audit..Oct 8 2015, 3:31 PM
cburroughs added a subscriber: cburroughs.Oct 8 2015, 5:28 PM
chad merged a task: T9533: Auto-create Maniphest tasks for concerns raised through Audit..Oct 17 2015, 3:24 AM
chad added subscribers: chad, • nick.udell.
eadler added a project: Restricted Project.Apr 14 2016, 4:45 AM
eadler added a subscriber: eadler.
eadler moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.Apr 17 2016, 6:24 PM
• kailei added a subscriber: • kailei.Oct 8 2016, 12:39 PM

How about this， it's solved or not?

chad mentioned this in Z1336: General Chat.Oct 31 2016, 4:40 PM
adam93 awarded a token.Nov 1 2016, 1:54 PM
adam93 added a subscriber: adam93.
adam93 added a comment.Nov 1 2016, 1:57 PM

Also interested in this. Our team plans on using the audit tool for code reviews, it be good if when a developer has made any suggested changes and re-committed that the old open audit could be closed/linked to the latest commit rather than having to comment on the old one with a link to the changed copy and just closing the old one manually.

epriestley removed a project: Differential.Dec 29 2016, 4:25 PM
epriestley moved this task from Backlog to Audit States on the Audit board.Jan 10 2017, 4:54 PM
epriestley added a subscriber: epriestley.Jan 25 2017, 8:29 PM

I'm going to merge this into T2393, where I'll discuss this shortly.

epriestley closed this task as a duplicate of T2393: Adds a "Needs Verification" state to Audits.Jan 25 2017, 8:29 PM
Phabricator · Built by Phacility