Following points should be taken into consideration : 1. Look into all individual commits 2. Look first into the files which have relevant changes. 3. We need to make sure that the PR's code should be working properly as per the new feature for which the PR is raised. 4. Code should not affect any previously existing feature. 3. Functions should not be too long. 4. There should not be redundant code. 5. Code should be clean i.e. there should be proper indendentation , no unnecessary blank lines , semicolon after each javascript statement. 6. Variables and function names should be meaningful . 7. If the application contains unit tests than the test coverage criteria should be met. for more details please refer the following link : What You Should Look Out For When You Review Pull Request