Code Review
Welcome to the programming.dev code review community! This is a place where you can review other people's code or request reviews on your own
Before posting please read the sections below (click on one to expand it)
What is a code review?
What is a code review?
A code review is a process where people who did not write a piece of code look over it to find errors, suggest improvements, and make sure it follows coding standards
When doing a code review look for these aspects:
- Correctness: Does the code behave as expected and meet the requirements?
- Readability: Is the code easy to understand and maintain?
- Efficiency: Are there more efficient ways to accomplish the same task?
- Best Practices: Does the code follow established coding standards and best practices?
- Security: Are there potential security vulnerabilities?
- Scalability: Will the code perform well as the application scales?
Post Guidelines
Guidelines
- Put the programming language you used for your code in square brackets ahead of your post title. e.g. [Python], [React], [C#], etc.
- Provide a link to a spot where people can view your code (e.g. github, etc.) in your post body
- In your post title put a small description of what the thing is that you want reviewed and then you can expand on that in more details if you want in your post body
Rules
Rules
Any code review requests you post after your first must be done after you have reviewed two other people's code (and then you need another two for each request after that as well).
Youre free to post your first code review request in this community though without needing to follow this
Credits
Credits
Icon base by Lorc under CC BY 3.0 with modifications to add a gradient
Wormhole
view the rest of the comments
Not really absolutely true. One can make a PR that only contains comments (not like in PRs, but
#
,//
,--
)That's true, but the person has to show which code to review. If it's the whole project or a single file, that's definitely possible, but multiple files or sections of code become more tricky then. "please review this file from line 40-100, that file from line 20-45, ..."