Review States and Transitions
The review work item documents the results of a designe or code review. The review work item must include detailed information about how the design or code met standards in areas of name correctness, code relevance, extensibility, code complexity, algorithmic complexity, and code security. If the review found that the design or code needed no changes, the review work item can be closed. If minor changes or major changes are needed, the active review work item is assigned to a developer to resolve. If minor changes are needed, the developer can close the review work item directly. If major changes are needed, a second review is required and the review work item is only closed if the second review passes successfully.
New
Review work items are created as the result of a design or code review.
New to Active
New |
A review is activated as a new review when it is first created. |
Active
An active review work item documents the results of a design or code review. Next steps are determined by the review meeting participants. The review work item will be closed if no changes are necessary. It will remain active and be assigned to the appropriate developer if changes are required. The developer will then resolve the review work item when the changes are completed.
Active to Resolved
Accepted w/Minor/Major Changes |
A review work item is resolved as Accepted w/minor/major changes when the developer has completed the identified changes in the design or code. |
Resolved
A resolved review work item indicates that the required changes, minor or major, have been completed. If the changes were major, a second review is required before the review work item can be closed. If the second review uncovers additional changes, the review work item is reactivated.
Resolved to Closed
Major Changes Complete |
A review work item is closed as Major Changes Complete when the second review determines that there are no remaining problems to address. |
Resolved to Active
Additional Changes Required |
A review work item is reactivated as Additional Changes Required when a second review indicates that additional problems were found in the design or code. The work item is assigned back to the responsible developer. |
Closed
A closed review work item indicates that the review is complete and all necessary changes, if any, have been completed. A closed review work item can be reactivated if it was determined that the review work item was closed in error.
Closed to Active
Closed in Error |
A review work item is reactivated as Closed in Error when it was determined that the work item was inadvertently closed. | |