Mailinglist Archive: yast-devel (78 mails)

< Previous Next >
[yast-devel] YaST Code Reviews
Hey developers,

Code reviews examine source code changes in order to find mistakes and
increase code quality. YaST code can also profit from systematic code reviews.
We should agree on a general way how to do code reviews for changes of YaST
code in the future.

From my perspective
- code reviews should be optional
- everyone should be permitted to read and review foreign code
- everyone should be able to ask for a code review for specific changes
- if you change somebody else's code you should ask for a review
- nobody should be forced to have his changes reviewed if that causes a
backlog (e.g. potential reviewer is busy or on vacation)

What's your opinion?

Cheers,
Thomas


--
Thomas Goettlicher
SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer,
HRB 16746 (AG Nürnberg)
Maxfeldstraße 5
90409 Nürnberg
Germany
--
To unsubscribe, e-mail: yast-devel+unsubscribe@xxxxxxxxxxxx
To contact the owner, e-mail: yast-devel+owner@xxxxxxxxxxxx

< Previous Next >
Follow Ups