search

rsanchez-wsu / fa15-ceg3120

11 stars 2 forks source link

Merge from master and clean up your team's code #26

Closed rsanchez-wsu closed 9 years ago

rsanchez-wsu commented 9 years ago

I have added the -Xlint option to the javac invocation for Ant and I have tightened up the CheckStyle checks. Please merge master into your team's branch, clean up all the warnings (paying attention to missing serialVersionUID, missing OverrideAnnotations, JavaDoc problems, etc.). For JavaDoc comments, please also remove any TODO markers or bogus comments and replace them with actual comments about what the class/method/whatever does. If the class/method/whatever is not yet implemented, then make comments about what it is intended to do once implemented. Please also remove code that has been commented out.

On Friday, I will be updating the build again to make any compiler and CheckStyle warnings fail the build.

jgotty777 commented 9 years ago

@rsanchez-wsu I think this is done. Am I allowed to close this?