-
Notifications
You must be signed in to change notification settings - Fork 4
Description
Artifact Under Review
VnV Report
Team Number for Team Doing the Review
Team 9
Description of Issue
In section 8, the team doesn't specify or differentiate between changes currently implemented and changes to be implemented. This is an important distinction that should be made. Furthermore, as it seems that most of these are phrased as changes already made, I suggest that the team consider future changes and add them to this section. Although you've passed the tests given, I believe you could reflect upon other testing methods or feedback given from supervisors, professors, and TAs to implement future changes.
Additionally, some of the columns are not readable due to text bleeding in from other columns (This was mentioned in another issue; however, I thought it should be brought to light here, as this issue is mainly reviewing section 8).