-
Notifications
You must be signed in to change notification settings - Fork 2
Code Review Checklist
Puvana Swatvanith edited this page Nov 14, 2020
·
10 revisions
- URLs
- Code have to navigate to the right pages.
- Views
- Method in views file can render to the right templates.
- Settings
- Settings file should have redirect for login, log out, and authentication.
- Templates
- Have the base template to reduce duplicate codes.
- Every feature in the template should be usable.
- Tests
- Code coverage is high(80 % or higher).
- Unittest can test most parts of the code.
- Use Flask8 corrections.
- Understandable comments.
- Use of good English, readable and correct grammar.