CI Requirements - Lessons Not Learnt?

[Ben Cooksley]

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

Regards,
Ben