CI Requirements - Lessons Not Learnt?
bcooksley at kde.org
Fri Jan 13 06:42:21 GMT 2017
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.
More information about the kde-core-devel