indygreg added a comment.
In https://phab.mercurial-scm.org/D1919#33462, @durin42 wrote: > I like this, and I'm fine with the backwards incompatible change in contrib. How do others feel? I'm fine with BC in contrib. One thing to bikeshed is if the `[phabricator]` section is the best place for credentials. We already have `[auth]` for storing credentials. But as long as we don't care about breaking BC, this patch is fine for now. Another thing to consider is reading things from `.arcconfig` files and reading the API token however Arcanist does it (as the comment in this file suggests). FWIW Mozilla will likely start leaning on this extension pretty soon. Look for a push from us to move it to core before the next release. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D1919 To: tom.prince, #hg-reviewers, durin42 Cc: indygreg, durin42, pulkit, mercurial-devel _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel