-
Identification of any functional issues. However, most of the review comments are unrelated to any types of functional defects.
-
Pointing out missing validation checks or alternate scenarios (including corner cases) where the current implementation may fail.
-
Suggestions regarding APIs to use, designs to follow, coding patterns, team coding conventions or best practices.
- “nit-picking issues” (e.g., indentation, comments, style, identifier naming, and typos). Resolution of nit-picking issues helps long-term project maintenance.