Review Checklist

Use this checklist to make sure your code review covers all the key points before merging changes. It's not fully comprehensive and may vary depending on the technology, but it's a solid template for most cases.

For Reviewers

Merge request title contains the correct issue number

The merge request is opened against the correct base branch

The MR description matches the changes in the code

The covered test cases are reasonable

There are no other obvious edge/special cases that should be explicitly covered

There is no apparent way to improve the performance and design of the new code

Is it simple?

Is it scalable?

Is it secure?

For Authors

My changes are documented, existing documentation is updated

My changes follow internal guidelines and industry-wide standards

My changes are covered by tests

Migrations are up to date with model changes

Database migrations are backward compatible

There are no conflicts with master branch

Corresponding issue is updated in project management platform (like Jira, ClickUp, etc.) with the link to this MR