Differential D16376

Add support for Differential Diffs to phid.lookup / PhabricatorObjectQuery
Needs ReviewPublic
Actions

Authored by dpkp on Aug 7 2016, 5:59 PM.

Tags

None

Referenced Files

	F14087043: D16376.diff
	Sat, Nov 23, 6:41 PM

	Unknown Object (File)
	Thu, Nov 21, 6:58 PM

	Unknown Object (File)
	Tue, Nov 19, 1:43 PM

	Unknown Object (File)
	Mon, Nov 18, 2:27 PM

	Unknown Object (File)
	Thu, Nov 14, 7:19 AM

	Unknown Object (File)
	Sun, Nov 10, 4:22 AM

	Unknown Object (File)
	Wed, Nov 6, 8:56 AM

	Unknown Object (File)
	Oct 24 2024, 7:31 AM

Subscribers

Details

Reviewers

joshuaspence
epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T11439: Retrieve Diff PHID via phid.lookup

Required Signatures

L28 Phacility Individual Contributor License Agreement

Summary

Test Plan

Tested phid.lookup(names=["Diff 15"])

Diff Detail

Repository

Branch

phid_lookup_diff

Lint

Lint Passed

Unit

Tests Skipped

Build Status

Buildable 13262
Build 16999: arc lint + arc unit

Event Timeline

dpkp updated this revision to Diff 39385.Aug 7 2016, 5:59 PM

dpkp retitled this revision from to Add support for Differential Diffs to phid.lookup / PhabricatorObjectQuery.

dpkp updated this object.

dpkp edited the test plan for this revision. (Show Details)

dpkp added reviewers: epriestley, joshuaspence.

dpkp added a task: T11439: Retrieve Diff PHID via phid.lookup.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 7 2016, 5:59 PM

Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript

Herald added a subscriber: Korvin. · View Herald Transcript

This is always super confusing because "Revisions" start with a "D" in their monogram, but a "diff" literally begins with the letter D while a revision does not.

I agree it's probably useful for this to exist but maybe we should come up with a better way of referring to a "diff" as opposed to a revision?

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

phid/

DifferentialDiffPHIDType.php

29 lines

Diff 39385

src/applications/differential/phid/DifferentialDiffPHIDType.php

Loading...