-
Notifications
You must be signed in to change notification settings - Fork 0
Team Deliverable #7: Code Review
2russellsmith edited this page Mar 26, 2015
·
1 revision
Our group had a code review that lasted 50 minutes and then analyzed our code review session for 10 minutes afterwards.
Analysis: It is really hard to stay on topic on a strict code review basis, because we all want to come up with solutions to the problems we were seeing instead of analyzing the code itself. We also found that we need to stick with looking at the code and not the actual implementation. Once we started looking at what the code is doing, it turned into a design and testing session instead of what it should have been. Overall we did get back on track and we were able to analyze the code and show places that it could be confusing for people who will come later to work on it.