Confirmed users
2,615
edits
| Line 15: | Line 15: | ||
; 1. Get UI buyoff : Doing this before worrying about the implementation avoids large rewrites because of basic design differences. | ; 1. Get UI buyoff : Doing this before worrying about the implementation avoids large rewrites because of basic design differences. | ||
; 2. Get basic code-level strategy agreement : Same benefit as 1. Even if you have a pretty good idea of how to attack a bug, the code owner may have big-picture (or little-picture) information that would suggest a different strategy. | ; 2. Get basic code-level strategy agreement : Same benefit as 1. Even if you have a pretty good idea of how to attack a bug, the code owner may have big-picture (or little-picture) information that would suggest a different strategy. | ||
; 3. Code with <u>ongoing</u> discussion : When you're wondering about some implementation issue, or see red flags (like having to copy | ; 3. Code with <u>ongoing</u> discussion : When you're wondering about some implementation issue, or see red flags (like having to copy/paste chunks of code, add weird dependencies, or violate abstractions), plunging ahead anyway is often the wrong thing to do. Instead, talk to a code owner or peer. Perhaps even post a patch fragment in the bug for input. | ||
; 4. Review | ; 4. Review | ||