CI Requirements - Lessons Not Learnt?

Martin Gräßlin mgraesslin at kde.org
Fri Jan 13 06:39:04 GMT 2017


Am 2017-01-13 03:10, schrieb Michael Pyne:
> On Thu, Jan 12, 2017 at 05:02:40PM +0100, Martin Gräßlin wrote:
>> Am 2017-01-12 08:32, schrieb Ben Cooksley:
>> > It would probably be a good idea to announce it for other developers
>> > to know about as well so they can sort their systems out.
>> 
>> that's what we have code review for :-)
> 
> No, code review isn't for every developer to review every single patch
> that comes across just to see if it introduces a dependency bump.  The
> cyclomatic complexity of that kind of review graph would be quite
> extreme ;).

I'm sure every member of a team knows how to inform the team. For Plasma
it would be code review. For other teams it might be a different way.

Cheers
Martin




More information about the kde-core-devel mailing list