Re: Gerrit supports inline edit feature now

Stephan Bergmann <sbergman -AT- redhat.com>
Thu, 18 Dec 2014 12:52:41 +0100

On 12/18/2014 12:44 PM, Michael Stahl wrote:

i disagree with your assessment of the feature.  it will make things
easier for reviewers,  because if you spot a trivial typo in a patch you
can just quickly fix it, you don't have to decide between downloading
the patch and editing it offline and pushing it manually, or adding a
comment and then waiting for the submitter to upload a new version of
the patch and get 4 email notifications about it and review it again.

...and the submitter unhelpfully rebasing the patch between uploads, soyou can't even easily find out whether indeed only the trivial typo wasfixed.

Context

Gerrit supports inline edit feature now · David Ostrovsky
- Re: Gerrit supports inline edit feature now · Miklos Vajna
  - Re: Gerrit supports inline edit feature now · Michael Meeks
    - Re: Gerrit supports inline edit feature now · Bjoern Michaelsen
      - Re: Gerrit supports inline edit feature now · Markus Mohrhard
        
        Re: Gerrit supports inline edit feature now · Bjoern Michaelsen
        
        Re: Gerrit supports inline edit feature now · Michael Stahl
        
        Re: Gerrit supports inline edit feature now · Stephan Bergmann

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.