Changes to our Git infrastructure

[Boudewijn Rempt]

On Mon, 5 Jan 2015, Aaron J. Seigo wrote:

> On Monday, January 5, 2015 22.26:24 Boudewijn Rempt wrote:
>> In short, what I meant is that as a tool to dicuss code changes,
>> Reviewboard is a poor thing. It facilitates nit-picking, which is
>> off-putting and useless, but at least gives the reviewer the feeling he's
>> done his job, while it fails at making it easy to discuss the why,
>> wherefore and how of a particular change.
>
> That is a development culture issue than no tool can fix.

Not fix, but a tool can encourage one way or another.

> Reviewboard is great
> for discussing changes in the hands of people who value that sort of
> interaction. It also can be used to deliver only nitpicking in a non-
> constructive manner. The difference in experience comes down to the what the
> people using it value.
>
> Where reviewboard is not good enough is in making it easy to quickly try the
> patch (bi-directional scm integration) ... but the commenting and discussion
> features are pretty much as good as it gets.
>
> -- 
> Aaron J. Seigo