<div dir="ltr">Hey,<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 18, 2014 at 11:47 AM, Bjoern Michaelsen <span dir="ltr"><<a href="mailto:bjoern.michaelsen@canonical.com" target="_blank">bjoern.michaelsen@canonical.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
On Thu, Dec 18, 2014 at 10:19:08AM +0000, Michael Meeks wrote:<br>
> -     // that sounds like a stupid idea<br>
> +     // actually this is really cool for small changes<br>
><br>
>       Also - we have various solutions to allow central compilation coming -<br>
> right.<br>
><br>
>       ;-)<br>
<br>
</span>Yes, inline editing is an important and cool thing. If our workflow cant handle<br>
that, we need to adapt it (e.g. more, earlier CI and unittests) not the other<br>
way around. ;)<br>
<br>
</blockquote><div><br></div><div>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.<br><br></div><div>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.<br><br></div><div>Regards,<br>Markus <br></div></div></div></div>