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: [email protected]
To contact the owner, e-mail: [email protected]

Reply via email to