Frockle/PeerReview2
Introduction
This exercise is similar to the first peer-review exercise. However rather than a physical meeting we will be conducting a tele-conference on 13th June 2007. It is a triangular round-robin where each team reviews the work of another, within a triplet. We are reviewing the work of DesignShare. In turn, we are being reviewed by ResourceBrowser. (NOTE: the wording in bold indicates points highlighted with respect to the first peer-review exercise.).
Main Exercise: DesignShare (conducted on April 24th 2007)
Pluses
Good project web-site which provides a nice introduction.
- Sensible code formatting and comments.
Being built as an Eclipse plugin, so effectively a standard build procedure (no scripts required) and then just Run As -> Eclipse Application to execute it.
Could not find a project blog - there is now a project blog (albeit sparsely populated one).
License is probably in the wrong place - should probably be placed at the top of the file above the package declaration. - this has been fixed.
Minuses
All the CVS commit messages are blank - helpful to have a bit more information, even if it's just "Added a bit more code ..."
- this was highlighted in the previous exercise. Whilst I (AO'C) could get the project to build and run from a cvs checkout as an Eclipse workbench project, I couldn't get it to run by dropping the plugin into a download of Reload (Ubuntu Linux 7.04 Feisty Fawn).
We suggested moving the location of the project web-site to a "nicer" location. It has moved to http://www.oyster-media.com/designshare/. If you go to http://www.oyster-media.com/ (the "root") you find a questionnaire. Just a bit disconcerting that's all. University branding would be nice
.
Footnotes
AO'C - Alexis O'Connor.