The goal for the next hour is to practice your technical code review skills and providing critical feedback. On the job, code reviews are huge part of being on a team and being able to accept and implement feedback is a vital skill.
Feedback should be positive, actionable and clear enough that anyone reading the issue could implement without any clarifications.
- Code Patterns - Identify coding patterns or practices that are detrimental to the code base or go against the majority of the code base (consistency). Are there too many global variables? Are you using .forEach instead of .map to create new arrays? Lots of if/else statements? Functions are too big?
- Style - Is your code consistently styled (spacing, naming conventions, file names)? Are you using a linter? Are you switching between ES5 and ES6?
- Communication - Are you writting concise commit messages? Are you using branches for small, complete pieces of functionality and have PR's with comments?