Testing
All specs are passing
New specs are well written, uses let helpers and before blocks
If Cucumber scenarios are provided in the ticket then they written, implemented and pass
Rails specific
Views have minimal logic, encourage the use of helpers
Controller actions should strive to be simple
Methods are object oriented, not excessive, and in the right models
General
Tabs are 2 spaces, no trailing whitespace
No inline styling
I8n code is in
508 compliance code is in
Find at least one line to praise
Commit msgs include ticket number reference
How to run rake task (if applicable)