Also +1 for that source not being review board On Wed, Sep 14, 2016 at 5:23 PM, Reed O'Brien <reed.obr...@canonical.com> wrote:
> Also +1 for a single source of truth. > > On Wed, Sep 14, 2016 at 1:20 PM, Rick Harding <rick.hard...@canonical.com> > wrote: > >> /me is always +1 on reducing the number of things we have to maintain and >> keeping things simpler. >> >> On Wed, Sep 14, 2016 at 4:04 PM Nate Finch <nate.fi...@canonical.com> >> wrote: >> >>> In case you missed it, Github rolled out a new review process. It >>> basically works just like reviewboard does, where you start a review, batch >>> up comments, then post the review as a whole, so you don't just write a >>> bunch of disconnected comments (and get one email per review, not per >>> comment). The only features reviewboard has is the edge case stuff that we >>> rarely use: like using rbt to post a review from a random diff that is not >>> connected directly to a github PR. I think that is easy enough to give up >>> in order to get the benefit of not needing an entirely separate system to >>> handle reviews. >>> >>> I made a little test review on one PR here, and the UX was almost >>> exactly like working in reviewboard: https://github.co >>> m/juju/juju/pull/6234 >>> >>> There may be important edge cases I'm missing, but I think it's worth >>> looking into. >>> >>> -Nate >>> -- >>> Juju-dev mailing list >>> Juju-dev@lists.ubuntu.com >>> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm >>> an/listinfo/juju-dev >>> >> >> -- >> Juju-dev mailing list >> Juju-dev@lists.ubuntu.com >> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailm >> an/listinfo/juju-dev >> >> > > > -- > Reed O'Brien > ✉ reed.obr...@canonical.com > ✆ 415-562-6797 > > > -- > Juju-dev mailing list > Juju-dev@lists.ubuntu.com > Modify settings or unsubscribe at: https://lists.ubuntu.com/ > mailman/listinfo/juju-dev > >
-- Juju-dev mailing list Juju-dev@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/juju-dev