GitHub -> Differential bridge for new contributors
Closed, DeclinedPublic
Actions

Assigned To

None

Authored By

	• Mattflaschen-WMF
	May 6 2014, 1:18 AM

Description

This task upstream contains useful background: Document one or two reasonable workflows for Github repositories

Upstream report: https://secure.phabricator.com/T589 (Closed/Declined, upstream will not maintain third-party importers).

A GitHub->Phabricator bridge (similar to T37497 for GitHub->Gerrit) would allow new contributors to make a change without installing arc.

Basically, new GitHub pull requests would get copied into Differential (the Phabricator code review tool).

See prior discussion at T127: Find way to use Differential with plain git (i.e.: without requiring arc).

Details

Reference: fl281

Related Objects

Mentioned In: T130507: Create labs project arcanist
T117388: Setup pull request -> differential bot for composer-merge-plugin
T715: GitHub -> Phabricator import system
T37497: Implement a way to bring GitHub pull requests into gerrit
T127: Find way to use Differential with plain git (i.e.: without requiring arc)
T409: Editing files and contributing changes via web
Mentioned Here: T715: GitHub -> Phabricator import system
T37497: Implement a way to bring GitHub pull requests into gerrit
T127: Find way to use Differential with plain git (i.e.: without requiring arc)

Event Timeline

qgil wrote on 2014-05-06 01:37:50 (UTC)

There seems to be something about this upstream (unfinished):

D8775: Integrate GitHub pull requests as Differential reviews

avive wrote on 2014-05-19 19:04:42 (UTC)

Until https://secure.phabricator.com/D8775 "Integrate GitHub pull requests as Differential reviews" can be re-written using future tools, Jenkins can run a client-side script to convert a PR into a Revision, post a relevant message on the PR, and maybe keep bringing updates from it.

I can try coding this up if it looks like something you could use.

mattflaschen wrote on 2014-07-26 01:46:47 (UTC)

Facebook has a bot for this. See https://github.com/facebook/hhvm/pull/3217 . Maybe they'd be willing to open source it.

aklapper wrote on 2014-08-28 21:13:18 (UTC)

In T281#12, @mattflaschen wrote:

Facebook has a bot for this. See https://github.com/facebook/hhvm/pull/3217 . Maybe they'd be willing to open source it.

Anybody in WMF who has contacts (e.g. via HHVM?) to make a friendly request to Facebook?

avive wrote on 2014-09-06 18:19:19 (UTC)

From the readme, this looks like a daemon that translates (and updates) branches to Revisions:

https://github.com/bloomberg/phabricator-tools/blob/master/doc/man/arcyd/arcyd.generated.txt

• flimport added a subscriber: Aklapper.Oct 1 2014, 11:03 PM

• flimport added a subscriber: Qgil.Oct 2 2014, 9:48 PM

• flimport added a subscriber: scfc.Oct 7 2014, 3:01 AM

• flimport added a subscriber: • Mattflaschen-WMF.Oct 8 2014, 11:00 PM

Qgil mentioned this in T409: Editing files and contributing changes via web.Oct 9 2014, 11:22 PM

Qgil mentioned this in T127: Find way to use Differential with plain git (i.e.: without requiring arc).Oct 10 2014, 9:10 AM

• flimport added a subscriber: revi.Oct 10 2014, 9:00 PM

asb subscribed.Nov 14 2014, 11:48 AM

Aklapper merged a task: T37497: Implement a way to bring GitHub pull requests into gerrit.Dec 1 2014, 12:13 PM

Aklapper mentioned this in T37497: Implement a way to bring GitHub pull requests into gerrit.

Aklapper added subscribers: Unknown Object (MLST), Eloquence, He7d3r and 13 others.

waldyrious awarded a token.Dec 3 2014, 4:59 PM

Meanwhile, a small project has been created upstream "Of or pertaining to integration with GitHub."

https://secure.phabricator.com/project/profile/582/

Qgil updated the task description. (Show Details)Jan 15 2015, 7:03 AM

Qgil set Security to None.

Qgil moved this task from Backlog to Need Discussion on the Phabricator (Upstream) board.Jan 16 2015, 10:17 AM

Qgil removed a project: Phabricator (Upstream).Feb 4 2015, 8:20 PM

JanZerebecki subscribed.Mar 3 2015, 5:45 PM

• Spage updated the task description. (Show Details)Mar 26 2015, 11:12 PM

Ricordisamoa subscribed.May 12 2015, 11:48 AM

Same reasoning from T715:

In T715#1620922, @greg wrote:

... this is a new feature (not something we have with Gerrit now) and thus not a blocker for the Gerrit-Migration project. Nor is it a blocker for the gitblit-deprecation (the gitblit-deprecation project should not be use for any and all code-hosting needs).

Adding Differential to track this going foward.

greg renamed this task from GitHub->Phabricator bridge for new contributors to GitHub -> Differential bridge for new contributors.Sep 9 2015, 5:42 PM

greg updated the task description. (Show Details)

greg removed a project: Gerrit-Migration.

JanZerebecki mentioned this in T715: GitHub -> Phabricator import system.Sep 10 2015, 10:42 AM

bd808 mentioned this in T117388: Setup pull request -> differential bot for composer-merge-plugin.Nov 2 2015, 11:36 PM

greg moved this task from To Triage to Backlog on the Differential board.Nov 4 2015, 7:12 PM

bd808 subscribed.Nov 29 2015, 9:22 PM

Chetane subscribed.Nov 29 2015, 9:40 PM

scfc mentioned this in T130507: Create labs project arcanist.Mar 20 2016, 7:21 PM