Project Manager: Is this function done?What's wrong with this exchange? To the developer and to the project manager, "done" means something rather different. To the developer in this case, done means: "I don't have to work on this piece of code any more (unless the tester tells me something is wrong)." The project leader is looking for a statement that the code is ready to ship.
Developer: Yes
Project Manager: So we can ship it?
Developer: Well, No. It needs to be tested, and I need to write some documentation, but the code works, really. I tested it... (pause) ...on my machine.
At its most basic level, a definition of Done creates a shared understanding of what it means to be finished, so everybody in the project means the same thing when they say "it's done". More subtly, the definition of Done is an expression of the team's quality standards. A more rigorous definition of Done will be associated with higher quality software. Generally the team will become more productive ("have a higher velocity") as their definition of Done becomes more stringent, because they will spend less time fixing old problems. Rework all but disappears.
So the Definition of Done should evolve as the project advances. Here is the first Definition of Done my team used when we started to develop the HappinessApp:
- Unit tests written and green
- Source code committed on server
- Jenkins built version and all tests green
- Code review completed (or pair-programmed)
- How-to-Demo verified before presentation to Product Owner
- Ok from Product Owner
This definition served us well until we started releasing our beta test results to customers, starting with me. I was are first customer. So we enhanced the definition of done as follows:
- Potentially releasable build available for download
- Summary of changes updated to include newly implemented features
- Inactive/unimplemented features hidden or greyed out (not executable)
- Unit tests written and green
- Source code committed on server
- Jenkins built version and all tests green
- Code review completed (or pair-programmed)
- How to Demo verified before presentation to Product Owner
- Ok from Product Owner
Very quickly, we discovered that an upgrade on an actual iPhone would overwrite existing user data. This was a bad thing (especially for a diary application!), so we added another point:
- Upgrade verified while keeping all user data intact.
- Potentially releasable build available for download
- Summary of changes updated to include newly implemented features
- Inactive/unimplemented features hidden or greyed out (not executable)
- Unit tests written and green
- Source code committed on server
- Jenkins built version and all tests green
- Code review completed (or pair-programmed)
- How to Demo verified before presentation to Product Owner
- Ok from Product Owner
This version held up pretty well through the next 8 sprints or so until we finished developing the beta version. We did experiment with two other points which were inspired by Release It, the guide to designing and deploying production-ready software:
- Design review -> list of how-could-this-feature-break failure cases (at least 3 unhappy paths)
- Identified failure cases covered in design decision or unit test
In retrospect, the Definition of Done and the engineering practices they imply have served us well. The product has been remarkably free of technical issues, so the team has achieved the proverbial "technical success." Next step for us is to get it out there and help in become an genuine success.
The Definition of Done is important to Scrum, but there are many facets to done-ness, of which the Definition of Done only covers a few. "What do you mean it's done? Only two little features have been implemented! Before we can call it done, all 150 features must be implemented!" I look forward to revisiting this topic in a future article, tentatively titled "The Three Faces of Done." In the mean time, my CSM Students (for whom this article was written) can explain it do you!

0 comments:
Post a Comment