Date: prev next · Thread: first prev next last
2014 Archives by date, by thread · List index


Hey,

On Thu, Dec 18, 2014 at 11:47 AM, Bjoern Michaelsen <
bjoern.michaelsen@canonical.com> wrote:

Hi,

On Thu, Dec 18, 2014 at 10:19:08AM +0000, Michael Meeks wrote:
-     // that sounds like a stupid idea
+     // actually this is really cool for small changes

      Also - we have various solutions to allow central compilation
coming -
right.

      ;-)

Yes, inline editing is an important and cool thing. If our workflow cant
handle
that, we need to adapt it (e.g. more, earlier CI and unittests) not the
other
way around. ;)


It is not that easy. This feature shifts some of the work from the
developer to the reviewer. IMHO our reviewers are already one of the
weakest link in our development process as reviewing is normally only done
by the people who have been working on the code for a long time and know
the code quite well.

I'd like to see solutions that shift work from these people and make it
easier for new reviewers to come in and not make their job more complex and
more demanding.

Regards,
Markus

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.