GitHub pull requests

Herbert Valerio Riedel hvriedel at gmail.com
Sun Oct 5 07:31:02 UTC 2014


On 2014-10-05 at 04:51:41 +0200, Richard Eisenberg wrote:
> I've just finished reading this: http://www.reddit.com/r/haskell/comments/2hes8m/the_ghc_source_code_contains_1088_todos_please/
>
> For better or worse, I don't read reddit often enough to hold a
> conversation there, so I'll ask my question here: Is there a way we
> can turn GitHub pull requests into Phab code reviews? I'm thinking of
> something like this:

My greatest worry about allowing GitHub PRs to the
github.com.ghc/ghc.git repo is that GitHub and Trac use the very same
`#[0-9]+` syntax tokens for referring to tickets and PRs, and trigger
actions as soon as they detect any commit using that token. In other
words, there's a namespace collisions (Luckily, Phabricator seems to
have been designed to be used in concert with an external Ticket
tracker, so it uses `[DT][0-9]+` to refer to code-revisions & tickets
respectively).

Morever, I'm also worrying this may become confusing to new
contributors, since we already have the Trav vs Phabricator confusion
about where to submit patches; if we also add GitHub PRs it'll just add
another item to be confused about where things ought to be
submitted. And the more PRs are added on github.com/ghc/ghc, the more it
may appear as if that is the encouraged way to submit them (even though
Phabricator+Trac is our currently targetted workflow)


What I'd suggest alternatively, since this what some of our contributors
are already doing instead of uploading patches:

  Teach Phabricator to allow to submit a URL to a commit (or branch) in a
  forked github.com/ghc/ghc repo, and create a code-revision out of that.


Cheers,
  hvr


More information about the ghc-devs mailing list