Closed Charleys488 closed 3 weeks ago
Scan your source code for useless code, comments, etc.
Assess whether methods and classes follow single use/responsibility.
There is almost always, no need for duplicated code.
Strive to be concise, write readable code with code style in mind.
Dense or complicated code is documented. Teach it to yourself.
Exceptions are handled and communicated properly (catch block, throw exception, descriptive logs)
Justify if a class/method is as many LOC (lines of code) as it needs to be.
Gauge the validity of your types, use dynamic/complex types as needed.
Scope of access should be as restrictive as possible.
Follow OOP, break big problems into smaller, simpler ones. Design methods/classes with this in mind.
You are the last line of defense.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
About this PR
55
How to review
Checklist
Scan your source code for useless code, comments, etc.
Assess whether methods and classes follow single use/responsibility.
There is almost always, no need for duplicated code.
Strive to be concise, write readable code with code style in mind.
Dense or complicated code is documented. Teach it to yourself.
Exceptions are handled and communicated properly (catch block, throw exception, descriptive logs)
Justify if a class/method is as many LOC (lines of code) as it needs to be.
Gauge the validity of your types, use dynamic/complex types as needed.
Scope of access should be as restrictive as possible.
Follow OOP, break big problems into smaller, simpler ones. Design methods/classes with this in mind.
You are the last line of defense.