Here at reinteractive, we care deeply about the quality of our code. We have many processes in place to ensure that the code we produce for our clients is of the highest quality, one of which is a mandatory code review. Before any feature makes it into staging, it must be reviewed by at least one, and preferably two, other developers.
What is a code review?
To quote Wikipedia, a code review is a systematic examination of source code. It is intended to find mistakes overlooked in the initial development phase, improving the overall quality of software.
What is a code review to me personally?
A code review is an exciting opportunity to learn, both for the requestor and for the reviewer. The exchange involved in getting a pull request accepted allows all parties to level up their skills. It is an opportunity to discuss problems, especially since you may find yourself with a similar problem in another project.
The hunt for a good solution fosters the feeling of teamwork. We are all in this together, and we all share the common goal of releasing high-quality software.
Finally, having all code reviewed by many different developers ensures consistency and improves the overall quality of the code.
Why Should I Care?
Because our valuable final product is bug-free features developed, delivered and accepted by the client, helping them achieve their goals within their budget.
Our Code Review Guidelines
PRIORITISE: - Make time to review. Taking the time to give a thorough code review benefits the requestor, the reviewer, and the client.
COMMON COURTESY: - Be nice. - Don't be personal. - Keep in mind that many programming decisions are opinions only. - If your opinion on how something should be done differs from that of the requestor, offer suggestions and discuss tradeoffs. - Avoid ownership. Remember we are a team. - Always review as if the requestor is a violent psychopath who knows where you know (adapted from John Woods famous quote)
WHAT TO FOCUS ON: - Discuss architecture - Talk about patterns - Use robots to review code style, so that PRs aren't just formatting-related comments
KEEP THESE KEY POINTS IN MIND: - Remember that sometimes you need to do something right now, not necessary right. For example, urgent security fixes. - Plan for the future but build for today.
CONVENTIONS: - Use @mentions to talk to someone. - When you're satisfied, approve the PR with a "LGTM 👍 ".
Dissecting Code With Ruby's caller Method
Handling deletes with Null Object Pattern in Ruby
reinteractive is Australia’s largest dedicated Ruby on Rails development company. We don’t cut corners and we know what we are doing.
We are an organisation made up of amazing individuals and we take pride in our team. We are 100% remote work enabling us to choose the best talent no matter which part of the country they live in. reinteractive is dedicated to making it a great place for any developer to work.
Webinars are our online portal for tips, tricks and lessons learned in everything we do. Make the most of this free resource to help you become a better developer.
The Ruby on Rails Installfest includes a full setup of your development environment and step-by-step instructions on how to build your first app hosted on Heroku. Over 1,800 attendees to date and counting.
The Ruby on Rails Development Hub is a monthly event where you will get the chance to spend time with our team and others in the community to improve and hone your Ruby on Rails skills.