**General** - [ ] Are all vars used somewhere? - [ ] Do the vars have properly cased, and meaningful names? - [ ] Style looks appropriate - [ ] No unnecessarily duplicated logic - [ ] Code intent is clear upon initial reading - [ ] Any code that isn't clear, but is needed, has an effective comment - [ ] Are method calls or attribute lookups every called on entities that could be undefined/null? - [ ] The functions can appropriately handle unexpected inputs. - [ ] Commented *code* has been removed (comments themselves are fine). - [ ] There are tests for the proposed functionality (if not, there's a good reason) - [ ] You've actually read the tests, and have a sense of what's being tested and what isn't. (or this is n/a) **Angular Specific** - [ ] No DOM manipulation is happening inside controllers - [ ] View logic is non-existant or minimal - [ ] Among the rendering logic, there are no good candidates to be extracted into a filter - [ ] Among the controller logic, there are no good candidates to be extracted into a service - [ ] Among all the logic, there are no good candidates to be extracted into a directive - [ ] Among the partials, there are no good candidates for extraction into an ng-include - [ ] No unneeded controllers were created. **Final Checks** - [ ] It has indeed passed build tests before getting merged - [ ] It does what the description says it does - [ ] Any technical debt has been added to the log.