HomePhorge

Add 'repositoryPHID' to 'differential.createrawdiff'

Description

Add 'repositoryPHID' to 'differential.createrawdiff'

Summary:
See https://github.com/facebook/phabricator/issues/596

Broadly, Facebook would like to bring Pull Requests from GitHub into Phabricator.

In the long term we can do this properly via Doorkeeper/Nuance, but that's probably a ways off. This seems like a reasonable low-budget compromise for now.

I'm a little hesitant to add a ton of parameters to this call, but repositoryPHID seems pretty reasonable, and is notable because it also controls default policies.

Test Plan:

  • Created a diff with no repositoryPHID.
  • Created a diff with a repositoryPHID.
    • Verified it carried over when the diff was used to create a revision.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: ptarjan, jamesgpearce, epriestley

Differential Revision: https://secure.phabricator.com/D9023

Details

Provenance
epriestleyAuthored on May 9 2014, 6:06 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP9d0d1ac42f8a: Speed up DiffusionBrowseFileController by removing call to array_merge
Branches
Unknown
Tags
Unknown

Event Timeline