0% found this document useful (0 votes)
78 views

Code Review Checklist

The document provides a revision history for a project specific code review checklist for KAP. It lists 4 versions from April 2019 to August 2021 with changes including adding checklists, version sheets and changing tab names. It also includes a checklist for reviewing code with categories like coding, database, structure and general and asks the reviewer to provide a status and remarks for each.
Copyright
© © All Rights Reserved
Available Formats
Download as XLSX, PDF, TXT or read online on Scribd
0% found this document useful (0 votes)
78 views

Code Review Checklist

The document provides a revision history for a project specific code review checklist for KAP. It lists 4 versions from April 2019 to August 2021 with changes including adding checklists, version sheets and changing tab names. It also includes a checklist for reviewing code with categories like coding, database, structure and general and asks the reviewer to provide a status and remarks for each.
Copyright
© © All Rights Reserved
Available Formats
Download as XLSX, PDF, TXT or read online on Scribd
You are on page 1/ 5

Revision History

Version Date

1.0 10-Apr-19

1.1 23-May-19

1.2 2-Feb-21

1.3 26-Aug-21
Revision History
Description

Initial Release

Added project spesific checklist, Added version sheet & Changed Tab names

As we are moving to KAP tool, created Unipro Equivalent "Code- Review" checklist.

Added defect related information in review checklist


Prepared / Modified By Approved By

Smita Mane Bharatkumar Mori

Vijay Sanap Bharatkumar Mori

Kirti Kale Bharatkumar Mori

Pradnya Joshi Bharatkumar Mori


Project OR R.0033563.002
Project Name FCA Enginieering IT
Document Project Specific Code Review Checklist for KAP
Low Severity Defect Count 0
Medium Severity Defect Count 0
High Severity Defect Count 0
WP Number WP_FBDD_0014

Review Checkpoints

Sr. No. Category Description

Does the code closes the connections to external resources like database connections, file handles, network
1 Coding
connections, etc?
2 Coding Has code handled transactions properly?

3 Database Have you used parameterised queries to avoid SQL Injections whenever applicable?

4 Structure Have you performed null check before using any objects?

5 Structure Is your code arranged in proper regions, namespaces and has proper indentations?

6 General Have you avoided hardcoded values and magic numbers?


7 General Is your code has proper comments whenever needed?
8 General Is code quality matches project specific code review guidelines?
Legends
Do not change Greyed out fields. They are either read-only or auto calculated.
Reviewer need to change only GREEN marked fields.

Reviewer Check
Defect Category Reviewer Remarks
Status
ase connections, file handles, network
Yes

Yes

ver applicable? Yes

Yes

ndentations? Yes

Yes
Yes
Yes

You might also like