canmove, Confirmed users
345
edits
No edit summary |
(→Ideas For How To Split Your Patch: "and then") |
||
Line 47: | Line 47: | ||
What you'll notice that all of these examples have in common is: | What you'll notice that all of these examples have in common is: | ||
* Every patch is ''functional''--that is, there are no patches that do not actually make some change that affects a user. There's no patch that just adds a library and doesn't use it. (Sometimes we actually ''do'' add a library in one bug and use it in another, but if that's the case, then we ''don't check in the library'' until the code that uses it is also ready for checkin.) | * Every patch is ''functional''--that is, there are no patches that do not actually make some change that affects a user. There's no patch that just adds a library and doesn't use it. (Sometimes we actually ''do'' add a library in one bug and then use it in another, but if that's the case, then we ''don't check in the library'' until the code that uses it is also ready for checkin.) | ||
* The small pieces are ''focused''--they work on one area of Bugzilla, or the smallest possible part that can be modified. | * The small pieces are ''focused''--they work on one area of Bugzilla, or the smallest possible part that can be modified. | ||
[[Category:Bugzilla|S]] | [[Category:Bugzilla|S]] |