User:Chani/WayOfThePlasma/Categories/Policy

Why must patches be peer-reviewed before they can be commited?

Conclusion

It is hard for new contributers to see all possible consequences of the changes they introduce. This has lead to long-term issues in earlier projects.

Original Text

Aaron Seigo

now, last week saw many unreviewed commits that actually resulted in rather unecessary issues in the code base; (...) people have a tendency to fall back to methods that got us kicker and kdesktop (namely: really useful programs that did a ton of a stuff but were at a point where they hit a brick wall as far as being able to take them further).

free-for-all does have a very negative impact on the code base. we tried that, and we moved to peer review because of what that resulted in.

Sources

panel-devel archive (2008-02)

Is it ok to make changes, even if they expose bugs in other parts of Plasma?

Conclusion

Absolutely. All known issues must be addressed. Visible bugs motivate people to fix the problem.

Original Text

Aaron Seigo

this is a good way to ensure that the issue is never addressed. by removing the obvious pain, you've swept the problem under the rug. it's not even a hard problem either, but this commit helps ensure the problem stays there.

please do not route around bugs like this in the future. =)

Sources

panel-devel archive (2008-02)