On 27 May 2015 at 09:29, Justin Bogner <mail at justinbogner.com> wrote:> Mehdi Amini <mehdi.amini at apple.com> writes: >> Hi Manuel, >> >> I like Phabricator for code review much much more than emails. Let me know how >> I can help (I’m not afraid of PHP). > > Chandler updated the llvm phabricator doc to point at what we're deploying: > > http://llvm.org/docs/Phabricator.html#status > > That'll lead you here: > > https://github.com/r4nt/llvm-reviews > https://github.com/r4nt/phabricatorFWIW: which branch is the deployed llvm one? llvm/master ? llvm/r4nt-master ? -- Eitan Adler
On Fri, May 29, 2015 at 1:54 AM Eitan Adler <lists at eitanadler.com> wrote:> On 27 May 2015 at 09:29, Justin Bogner <mail at justinbogner.com> wrote: > > Mehdi Amini <mehdi.amini at apple.com> writes: > >> Hi Manuel, > >> > >> I like Phabricator for code review much much more than emails. Let me > know how > >> I can help (I’m not afraid of PHP). > > > > Chandler updated the llvm phabricator doc to point at what we're > deploying: > > > > http://llvm.org/docs/Phabricator.html#status > > > > That'll lead you here: > > > > https://github.com/r4nt/llvm-reviews > > https://github.com/r4nt/phabricator > > FWIW: which branch is the deployed llvm one? llvm/master ? > llvm/r4nt-master ? >Ah, sorry, it's r4nt-staging (for completely bogus reasons, I need to change that). Thanks! /Manuel> > > > > > -- > Eitan Adler > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20150529/d0fae9f1/attachment.html>
On 29 May 2015 at 01:00, Manuel Klimek <klimek at google.com> wrote:> On Fri, May 29, 2015 at 1:54 AM Eitan Adler <lists at eitanadler.com> wrote: >> >> On 27 May 2015 at 09:29, Justin Bogner <mail at justinbogner.com> wrote: >> > Mehdi Amini <mehdi.amini at apple.com> writes: >> >> Hi Manuel, >> >> >> >> I like Phabricator for code review much much more than emails. Let me >> >> know how >> >> I can help (I’m not afraid of PHP). >> > >> > Chandler updated the llvm phabricator doc to point at what we're >> > deploying: >> > >> > http://llvm.org/docs/Phabricator.html#status >> > >> > That'll lead you here: >> > >> > https://github.com/r4nt/llvm-reviews >> > https://github.com/r4nt/phabricator >> >> FWIW: which branch is the deployed llvm one? llvm/master ? >> llvm/r4nt-master ? > > > Ah, sorry, it's r4nt-staging (for completely bogus reasons, I need to change > that).Thanks for the info - I wanted to take a look at what changes llvm had made. While looking I noticed that you have an extension requiring specific mailing lists be included with every diff: just a head's up that you will want to change that extension soon since mailing lists are becoming users, similar to bots. See https://secure.phabricator.com/T8387 -- Eitan Adler