[PATCH] phabricator: drop support for the deprecated `phabricator.token` config

[Augie Fackler]

> On May 12, 2018, at 11:46 AM, Matt Harbison <mharbison72 at gmail.com> wrote:
> 
> On Sat, 12 May 2018 11:18:18 -0400, Augie Fackler <raf at durin42.com> wrote:
> 
>> 
>> 
>>> On May 11, 2018, at 9:31 PM, Matt Harbison <mharbison72 at gmail.com> wrote:
>>> 
>>> I dabbled with Reviewboard this week, but I like how this can download a series, remember what review a commit is associated with, etc.  Bundling it would be nice for juggling different versions of Mercurial.  But Augie's comment on the bug tracker[1] made me think that maybe it's too hard to support properly.
>> 
>> I haven’t used reviewboard in a while, but Mozilla is moving from RB to Phab, which was enough to convince me to not go back and look at RB again.
> 
> OK, good info, thanks.  I really don't have any experience with either, other than the occasional review here.  Another group at my company uses RB, so I'm trying to figure out if there are technical benefits to Phab, without sinking a bunch of time into it.  (I'd consider a bundled extension that doesn't need external maintenance, plus evolve support, to be a huge benefit.)
> 
> Have there been administrative or user issues?  It seems to be running pretty smoothly, looking at it from a distance.

The biggest user issue is (to my knowledge) a peculiarity of our review process: we expect submissions to just be sent in general to the list, and reviewers to pick them up. junw built us a custom UI that helps a fair amount, visible at https://phab.mercurial-scm.org/yadda/, code at https://bitbucket.org/quark-zju/phab-yadda/

I’d be absurdly happy if we can move the extension to hgext from contrib. I’m a little busy with family right now, but I can promise review time...