Changes

Jump to navigation Jump to search
Line 17: Line 17:  
* Requirement to use [[Activity_Team/Sugar_Lint|sugar-lint]] (or so) in git pre-commit. [[User:Alsroot|alsroot]] 07:59, 25 October 2010 (EDT)
 
* Requirement to use [[Activity_Team/Sugar_Lint|sugar-lint]] (or so) in git pre-commit. [[User:Alsroot|alsroot]] 07:59, 25 October 2010 (EDT)
 
* Increase a level of trust within core team, make patch reviewing workflow more clear:
 
* Increase a level of trust within core team, make patch reviewing workflow more clear:
** Any submitter, to a particular core project, might be a reviewer (we either should trust all these people or drop them from committer list) [[User:Alsroot|alsroot]] 07:59, 25 October 2010 (EDT)
+
** Any committer, to a particular core project, might be a reviewer (we either should trust all these people or drop them from committer list) [[User:Alsroot|alsroot]] 07:59, 25 October 2010 (EDT)
 
** To accept ''regular'' (reviewer decides whether patches are regular or not, of course, better to discuss it with other committers if there are doubts) patches, only one reviewer is needed [[User:Alsroot|alsroot]] 07:59, 25 October 2010 (EDT)
 
** To accept ''regular'' (reviewer decides whether patches are regular or not, of course, better to discuss it with other committers if there are doubts) patches, only one reviewer is needed [[User:Alsroot|alsroot]] 07:59, 25 October 2010 (EDT)
 
** Non-regular patches should be formed in [[Features]]. These patches might take several tens of K. This is by design; that it is hard not only to develop, but also to review and it should involve several committers for a reasonable amount of time. [[User:Alsroot|alsroot]] 09:28, 25 October 2010 (EDT)
 
** Non-regular patches should be formed in [[Features]]. These patches might take several tens of K. This is by design; that it is hard not only to develop, but also to review and it should involve several committers for a reasonable amount of time. [[User:Alsroot|alsroot]] 09:28, 25 October 2010 (EDT)

Navigation menu