- Does the code work?
- Description of the project status is included.
- Code is easily understand.
- Code is written following the coding standarts/guidelines (React in our case).
- Code is in sync with existing code patterns/technologies.
- DRY. Is the same code duplicated more than twice?
sezeresim
/ gist:01d0d3fa1806e184329f1427cdeec7c2
Created
February 10, 2021 22:59
— forked from justinhillsjohnson/gist:5503121
code-review-checklist
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| General | |
| 1. Site uses a cache buster for expiring .js, .css, and images | |
| 2. JavaScript and CSS is minified and concatenated into logical groupings | |
| 3. Images have been optimized by ImageOptim (http://imageoptim.com/) | |
| Markup | |
| 1. Code does not contain inline JavaScript event listeners |
sezeresim
/ review-checklist.md
Created
February 10, 2021 22:52
— forked from bigsergey/review-checklist.md
Front-end Code Review Checklist