Confirmed users
2,615
edits
| Line 18: | Line 18: | ||
; 4. Review | ; 4. Review | ||
Because everyone is better sync throughout, much of the feedback being requested requires less time on the part of the code owner than dealing with one big patch of the end of the process. In other words, even though there are more points at which blocking can happen, the total amount of time actually spent in a blocked state is often less. | Because everyone is in better sync throughout, much of the feedback being requested requires less time on the part of the code owner than dealing with one big patch of the end of the process. In other words, even though there are more points at which blocking can happen, the total amount of time actually spent in a blocked state is often less. | ||
Note that these are just guidelines that are subject to judgement calls, not hard and fast rules. Five-line patches generally don't need this level of coordination. Sometimes it makes sense to discuss UI design and code-level strategy together. A quick prototype may occasionally be useful. Implementation will sometimes turn up issues that effect the design. Furthermore, these guidelines are not a magic bullet: occasionally patches end up needing to be re-written | Note that these are just guidelines that are subject to judgement calls, not hard and fast rules. Five-line patches generally don't need this level of coordination. Sometimes it makes sense to discuss UI design and code-level strategy together. A quick prototype may occasionally be useful. Implementation will sometimes turn up issues that effect the design. Furthermore, these guidelines are not a magic bullet: occasionally patches end up needing to be re-written, and sometimes bugs still bottleneck on one person's time. But in general, they help a fair bit. | ||
== Key success strategies == | == Key success strategies == | ||