Review Guide

In this guide we have written a checklist of what you should be looking at as an reviewer of code.

Libraries to avoid

Some libraries (that we have on the classpath) should be avoided if possible.

• Apache Commons Lang
  • org.apache.commons.lang (try to use com.google.guava classes instead)
• Apache Commons IO
  • org.apache.commons.io (try to use com.google.guava classes instead)

Deprecated API usages

Look out for deprecated 3rd party API usages. We should avoid using deprecated methods and classes if it can be avoided. Here's a list of some wide used deprecated classes/methods that must be avoided:

• junit.framework.Assert (use org.junit.Assert)