CI Requirements - Lessons Not Learnt?

Ben Cooksley bcooksley at
Fri Jan 13 06:42:21 GMT 2017

On Fri, Jan 13, 2017 at 7:39 PM, Martin Gräßlin <mgraesslin at> wrote:
> 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.

Please remember that your software is built by others outside the Plasma team.
Just letting the members of the Plasma team know isn't very considerate to them.

> Cheers
> Martin


More information about the kde-core-devel mailing list