[prev in list] [next in list] [prev in thread] [next in thread] 

List:       calligra-devel
Subject:    Re: Phabricator: All repositories registered - upcoming workflow changes
From:       Milian Wolff <mail () milianw ! de>
Date:       2017-02-01 8:48:58
Message-ID: 12499543.iBb1rC7C5K () milian-kdab2
[Download RAW message or body]

On Tuesday, January 31, 2017 7:56:52 PM CET Ben Cooksley wrote:
> On Tue, Jan 31, 2017 at 11:36 PM, René J.V. Bertin <rjvbertin@gmail.com> 
wrote:
> > On Sunday January 29 2017 08:32:21 Ben Cooksley wrote:
> > 
> > Hi,
> 
> Hi Rene,
> 
> > >From this point forward, communities should be moving away from
> >>
> >>Reviewboard to Phabricator for conducting code review. Sysadmin will
> >>be announcing a timeline for the shutdown of Reviewboard in the near
> >>future.
> >>
> > I hope that shutdown doesn't mean complete disconnect; it would probably
> > be a loss of as-yet unknown importance if all code reviews become
> > unavailable.
> > 
> > I'll miss ReviewBoard. Phabrithingy may be more powerful and versatile,
> > but RB had its advantages too which could be why it's still being used
> > (quite a lot, as far as I can see) and hasn't been integrated with KDE's
> > own IDE yet.
>
> It will be a complete shutdown of Reviewboard - we'll be archiving it
> in the event for some reason it becomes necessary to access the data
> it stores.

This is a *very* bad idea!

- Quite some commits will lose some extended history from the review comments
- What about the not-yet-merged changes?

If at all possible, please find a way to keep this site alive in a read-only 
mode.
 
> In most cases mailing lists should have the history of reviews in
> their archives, so those will continue to be accessible through list
> archives in the long run.

And how do you find the corresponding mail archive thread based on a 
reviewboard URL? Will there be auto-forwarding in-place?

Bye

-- 
Milian Wolff
mail@milianw.de
http://milianw.de
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic