Hi team,

This Monday (March 27, 2006) at noon in POST 307, we will review HACK-628 (CLI SDT Evolution). To perform this review:

(a) Install/update the Eclipse Jupiter plugin.
(b) Install/update and enable the Hackystat Jupiter sensor.
(c) Switch to SVN branch 'branches/HACK-628'

The Jupiter Review ID "HACK-628" has been defined in two modules: hackySdt_Cli and hackySensor_Cli. Please review the files indicated by the Review ID in those modules.

I am particularly concerned about the following issues:

* Has the evolution been performed correctly? Do you feel the resulting set of required and optional fields are appropriate?

* Is the Docbook documentation for the CLI SDT clear? (You might want to invoke 'ant hackyCore_Build.buildDocBook' to generate the formatted version).

After finishing your review, please be sure to SVN commit your changes. Note that you will need to SVN 'add' your files (and potentially the review/ directory) before doing the commit.

---------------------------------------
Supplemental links and resources

The Hackystat Code Review Guidelines:
* http://hackydev.ics.hawaii.edu/hackyDevSite/doc/Review.html

The Elements of Hacky Style:
* http://hackydev.ics.hawaii.edu/hackyDevSite/doc/EHS.html

Jupiter Code Review Tool documentation:
* http://csdl.ics.hawaii.edu/Tools/Jupiter/Core/doc/UsersGuide.html

----------------------------------------

Thanks for reviewing the code!

Mike

Reply via email to