2021-08-01 13:15:57 +00:00
|
|
|
|
|
|
|
|
<!-- markdownlint-disable-next-line MD041 -->
|
2018-08-23 17:23:45 +00:00
|
|
|
##### Project
|
2018-08-24 10:30:24 +00:00
|
|
|
|
2018-08-23 17:23:45 +00:00
|
|
|
::[PROJECT-NAME](Home)
|
2018-08-14 12:48:17 +00:00
|
|
|
|
2018-08-24 10:30:24 +00:00
|
|
|
##### Release Number
|
|
|
|
|
|
2018-08-14 12:48:17 +00:00
|
|
|
::X.Y.Z
|
|
|
|
|
|
2018-08-24 10:30:24 +00:00
|
|
|
##### Location of Meeting
|
|
|
|
|
|
2018-08-14 12:48:17 +00:00
|
|
|
::LOCATION, BUILDING, ROOM
|
|
|
|
|
|
2018-08-24 10:30:24 +00:00
|
|
|
##### Date and Time of Meeting
|
|
|
|
|
|
2018-08-14 12:48:17 +00:00
|
|
|
::YEAR/MONTH/DAY HH:MM
|
|
|
|
|
|
2018-08-24 10:30:24 +00:00
|
|
|
##### Attendees
|
2018-08-14 12:48:17 +00:00
|
|
|
|
2018-08-23 17:23:45 +00:00
|
|
|
- ::PERSON-NAME
|
|
|
|
|
- ::PERSON-NAME
|
|
|
|
|
- ::PERSON-NAME
|
|
|
|
|
- ::PERSON-NAME
|
2018-08-14 12:48:17 +00:00
|
|
|
|
2018-08-23 17:23:45 +00:00
|
|
|
##### Related Documents
|
2018-08-24 10:30:24 +00:00
|
|
|
|
2018-08-14 22:37:52 +00:00
|
|
|
- [QA Plan](QA-Plan) > Review Meeting Notes
|
|
|
|
|
- [Review Meeting Checklists](Review-Meeting-Checklists)
|
2018-08-14 12:48:17 +00:00
|
|
|
|
|
|
|
|
---
|
|
|
|
|
|
|
|
|
|
### Documents and Code Reviewed at this Meeting
|
|
|
|
|
|
2021-08-01 13:15:57 +00:00
|
|
|
- ::[Feature list section of requirements](#tbd)
|
|
|
|
|
- ::[Multi-user section of requirements](#tbd)
|
2018-08-24 10:30:24 +00:00
|
|
|
- ::[Hello.java](/source/browse/PROJECT-NAME/src/Hello.java)
|
|
|
|
|
- ::[HelloStream.java](/source/browse/PROJECT-NAME/src/HelloStream.java)
|
|
|
|
|
- ::[HelloPanel.java](/source/browse/PROJECT-NAME/src/HelloPanel.java)
|
2018-08-14 12:48:17 +00:00
|
|
|
|
|
|
|
|
### Meeting Minutes
|
|
|
|
|
|
|
|
|
|
*TODO: Fill in the information above and write a brief summary of what
|
|
|
|
|
happened at the meeting. The text below is just a sample.*
|
|
|
|
|
|
|
|
|
|
::We started on time with 3 members and the other team members arrived
|
|
|
|
|
shortly thereafter.
|
|
|
|
|
|
|
|
|
|
::In the first hour of the review meeting, we went over issues that the
|
|
|
|
|
team had entered individually. Most of the issues raised questions about
|
|
|
|
|
the error handling capability of Hello.java, but there were also issues
|
|
|
|
|
about multi-user robustness requirements and UI.
|
|
|
|
|
|
|
|
|
|
::In the second hour, we read through all of the methods of
|
|
|
|
|
HelloStream.java. All team members agreed that they understood each
|
|
|
|
|
method and thought is was correct before we moved on to the next method.
|
|
|
|
|
We raised issues of consistency between the constructors in that class
|
|
|
|
|
and other classes. The checklist was helpful in identifying the fact
|
|
|
|
|
that we needed to implement hash() and equals(). All raised issues were
|
|
|
|
|
entered into the issue tracker and marked with the word "review".
|
|
|
|
|
|
|
|
|
|
::Although, some issues were found, we do not feel that Hello.java and
|
|
|
|
|
HelloStream.java need to be inspected again. HelloPanel was not
|
|
|
|
|
inspected, it still needs review.
|
|
|
|
|
|
|
|
|
|
### Action Items
|
|
|
|
|
|
|
|
|
|
::All defects and tasks identified in this review are being tracked in the
|
|
|
|
|
issue tracker.
|
