Re:Collaboration not Derision in the Node Community
by Lazare on Sunday December 01, 2013 @01:08AM
(#6826345)
Technically, he was in the right.
Politically, reverting Isaac's commit (and especially doing so with a passive aggressive "Isaac, consider yourself chided." commit message) was clearly a mistake.
Re:Collaboration not Derision in the Node Community
by GrinningFool on Sunday December 01, 2013 @12:29PM
(#6828199)
A mistake, but only in context of the larger issue. In the context of a reverted commit, it was a fairly laid-back way of saying "you shouldn't have done this direct merge and you probably knew better".
Ultimately, it looks like Ben was fairly impartial about the whole thing - he didn't accept it because it wasn't a significant enough change to merit accepting. He tried to revert the acceptance because the acceptance had been done without approval.