On 8/22/2014 5:22 AM, Graham Bloice wrote:
On 22 August 2014 10:18, Thomas Wiens <th.wi...@gmx.de
<mailto:th.wi...@gmx.de>> wrote:
I've got another question to working on the comments in the review
system:
Is it good style to push every fixed comment as a single commit, or
should I work on all comments, and commit them together as once, with
multiple comments?
I've looked into older git comments in the review system, but did't
found a nice review process with commits, to look what style you prefer.
IMHO, I'd prefer to see reviewer lead changes in one lump as diffing
each patch set could be tedious. I'm basically reviewing the final
patch as will be merged to master. Others may have a different view.
--
Graham Bloice
+1
___________________________________________________________________________
Sent via: Wireshark-dev mailing list <wireshark-dev@wireshark.org>
Archives: http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:wireshark-dev-requ...@wireshark.org?subject=unsubscribe