How to Make Good Code Reviews Better

How to Make Good Code Reviews Better

I have been doing day-to-day code reviews for over a decade now. The benefits of code reviews are plenty: someone spot checks your work for errors, they get to learn from your solution, and the collaboration helps to improve the organization’s overall approach to tooling and automation. If you’re not currently doing code reviews in your organization, start now. It’ll make everyone a better engineer.

Plenty of people and organizations have shared their code review best practices and what the definition of good code reviews mean to them. Guides  from Google,  the SmartBear team, and engineer Philipp Hauerare all excellent reads. Below is my personal take on what good code reviews look like and how to make them even better at the team and organizational level. This is in the context of the tech environment I have been working at – currently at Uber, and before that at Skype/Microsoft and Skyscanner.

Good code reviews are the bar that all of us should strive for. They cover common and easy to follow best practices that any team can get started with, while ensuring high-quality and helpful reviews for the long term.

Better code reviews are where engineers keep improving how they do code reviews. These code reviews look at the code change in the context of the codebase, of who is requesting it and in what situation. These reviews adjust their approach based on the context and situation. The goal not only being a high-quality review, but also to help the developers and teams requesting the review to be more productive.

Areas Covered by the Code Review

Good code reviews look at the change itself and how it fits into the codebase. They will look through the clarity of the title and description and “why” of the change. They cover the correctness of the code, test coverage, functionality changes, and confirm that they follow the coding guides and best practices. They will point out obvious improvements, such as hard to understand code, unclear names, commented out code, untested code, or unhandled edge cases. They will also note when too many changes are crammed into one review, and suggest keeping code changes single-purposed or breaking the change into more focused parts.

Better code reviews look at the change in the context of the larger system, as well as check that changes are easy to maintain. They might ask questions about the necessity of the change or how it impacts other parts of the system. They look at abstractions introduced and how these fit into the existing software architecture. They note maintainability observations, such as complex logic that could be simplified, improving test structure, removing duplications, and other possible improvements. Engineer Joel Kemp describes great code reviews as  a contextual pass following an initial, light pass.

Tone of the Review

The tone of code reviews can greatly influence morale within teams. Reviews with a harsh tone contribute to a feeling of a hostile environment with their microaggressions. Opinionated language can turn people defensive, sparking heated discussions. At the same time, a professional and positive tone can contribute to a more inclusive environment. People in these environments are open to constructive feedback and code reviews can instead trigger healthy and lively discussions.

Good code reviews ask open-ended questions instead of making strong or opinionated statements. They offer alternatives and possible workarounds that might work better for the situation without insisting those solutions are the best or only way to proceed. These reviews assume the reviewer might be missing something and ask for clarification instead of correction.

Better code reviews are also empathetic. They know that the person writing the code spent a lot of time and effort on this change. These code reviews are kind and unassuming. They applaud nice solutions and are all-round positive.

Approving vs Requesting Changes

Once a reviewer completes their review, they can either mark it approved, block the review with change requests, or not set a specific status, leaving it in a “not yet approved” state. How reviewers use the approve and request changes statuses is telling of the code reviews.

Good code reviews don’t approve changes while there are open-ended questions. However, they make it clear which questions or comments are non-blocking or unimportant, marking them distinctively. They are explicit when approving a change – e.g. adding a thumbs up comment like “looks good!”. Some places use acronyms like LGTM—these also work, but be aware that newcomers could misinterpret these insider acronyms for something else. Good code reviews are equally explicit when they are requesting a follow-up, using the code review tool or team convention to communicate this.

Better code reviews are firm on the principle but flexible on the practice: sometimes, certain comments are addressed by the author with a separate, follow-up code change. For changes that are more urgent than others, reviewers try to make themselves available for quicker reviews.

From Code Reviews to Talking to Each Other

Code reviews are usually done asynchronously and in writing through a code review tool. This is usually out of convenience, to enable remote code reviews, and to allow multiple people to review the same code change. But when is it time to stop using the tool—however good it might be—and start talking face to face about the code?

Good code reviews leave as many comments and questions as are needed. If the revision does not address all of them, they will note those as well. When the conversation gets into a long back-and-forth, reviewers will try to switch to talking to the author in-person instead of  burning more time using the code review tool.

Better code reviews will proactively reach out to the person making the change after they do a first pass on the code and have lots of comments and questions. These people have learned that they save a lot of time, misunderstandings, and hard feelings this way. The fact that there are many comments on the code indicates that there is likely some misunderstanding on either side. These kinds of misunderstandings are easier identified and resolved by talking things through.


Nitpicks are are unimportant comments, where the code could be merged without even addressing these. These could be things like variable declarations being in alphabetical order, unit tests following a certain structure, or brackets being on the same line.

Good code reviews make it clear when changes are unimportant nitpicks. They usually mark comments like these distinctively, adding the “nit:” prefix to them. Too many of these can become frustrating and take the attention away from the more important parts of the review, so reviewers aim to not go overboard with these.

Better code reviews realize that too many nitpicks are a sign of lack of tooling or a lack of standards. Reviewers who come across these frequently will look at solving this problem outside the code review process. For example, many of the common nitpick comments can be solved via automated linting. Those that cannot can usually be resolved by the team agreeing to certain standards and following them—perhaps even automating them, eventually.

Code Reviews for New Joiners

Starting at a new company is overwhelming for most people. The codebase is new, the style of programming is  different than before, and people review your code very differently. So should code reviews be gentler for new starters, to get them used to the new environment, or should they keep the bar just as high, as it is for everyone else?

Good code reviews use the same quality bar and approach for everyone, regardless of their job title, level or when they joined the company. Following the above, code reviews have a kind tone, request changes where needed, and will reach out to talk to reviewers when they have many comments.

Better code reviews pay additional attention to making the first few reviews for new joiners a great experience. Reviewers are empathetic to the fact that the recent joiner might not be aware of all the coding guidelines and might be unfamiliar with parts of the code. These reviews put additional effort into explaining alternative approaches and pointing to guides. They are also very positive in tone, celebrating the first few changes to the codebase that the author is suggesting.

Cross-Office, Cross-Time Zone Reviews

Code reviews get more difficult when reviewers are not in the same location. They are especially challenging when reviewers are sitting in very different time zones. I have had my fair share of these reviews over the years, modifying code owned by teams in the US and Asia, while being based in Europe.

Good code reviews account for the time zone difference when they can. Reviewers aim to review the code in the overlapping working hours between offices. For reviews with many comments, reviewers will offer to chat directly or do a video call to talk through changes.

Better code reviews notice when code reviews repeatedly run into timezone issues and look for a systemic solution, outside the code review framework. Let’s say a team from Europe is frequently changing a service that triggers code reviews from the US-based owner of this service. The system-level question is why these changes are happening so frequently. Are the changes done in the right codebase or should another system be changed? Will the frequency of changes be the same or go down over time? Assuming the changes are done in the right codebase and the frequency will not go down, can the cross-office dependency be broken in some way? Solutions to these kinds of problems are often not simple and could involve refactoring, creating of new services/interfaces or tooling improvements. But solving dependencies like this will make the life of both teams easier and their progress more efficient for the long term, meaning the return on investment is often quite impressive.

Organizational Support

The way companies and their engineering organizations approach code reviews is a big element of how efficient they can be. Organizations that view them as unimportant and trivial end up investing little in making reviews easier. In cultures like this, it might be tempting to just do away with code reviews entirely. Engineers advocating for doing better code reviews might feel isolated, without support from above and eventually give up. The result is an organization where problems continue to repeat and compound upon themselves.

Organizations with good code reviews ensure that all engineers take part in the code review process—even those that might be working on solo projects. They encourage raising the quality bar, and teams facilitate healthy discussions on code review approaches both at the team and org level. These companies often have code review guides for larger codebases that engineers initiated and wrote. Organizations like this recognise that code reviews take up a good chunk of engineers’ time. Many of these companies will add code reviews as expectations to the developer job competencies, expecting senior engineers to spend a larger chunk of their time reviewing the code of others.

Organizations with better code reviews have hard rules around no code making it to production without a code review—just as business logic changes don’t make it to production without automated tests. These organizations have learned that the cost of cutting corners is not worth it; instead, they have processes for expedited reviews for urgent cases. These organizations invest in developer productivity, including working continually to develop more efficient code reviews and tooling improvements. Helpful engineering executives don’t need convincing on the benefits of code reviews and other engineering best practices. Instead, they support initiatives on better tooling or more efficient code review processes that come from teams.

When people come across reviews that feel hostile, they feel they can speak up and have support all-round to resolve the issue. Senior engineers and managers consider code reviews that are not up to the bar just as much of an issue as sloppy code or poor behavior. Both engineers and engineering managers feel empowered to improve how code reviews are done.

Start With Good, Make it Better

Good code reviews already have lots of good effort going into them. They do a thorough review of the change itself, avoid being opinionated with the tone of comments, and make nitpicks clear. They maintain a consistent bar, regardless of who is requesting the review and try to make cross-time zone reviews less painful by paying additional attention to these. Organizations that have good reviews ensure that every developer regularly receives and does code reviews. This is already a high bar—but if you get here, don’t stop. Code reviews are one of the best ways to improve your skills, mentor others, and learn how to be a more efficient communicator.

Get to better code reviews by continuously improving on the details, but also start looking at changes at a high level as well. Be empathetic in the tone of comments and think of ways outside the code review process to eliminate frequent nitpicks. Make code reviews especially welcoming for new starters and look for systemic solutions for painful cross-time zone reviews. Organizations that are forward-looking encourage investing in tooling and process improvements to make code reviews better, reaping more of the benefits.

@Credit from

[Codester] – Aeon – Domain for Sale HTML Template – Freebies Download

[Codester] – Aeon – Domain for Sale HTML Template – Freebies Download

A professional site template that can be quickly configured and put up to help sell a domain name.

Live Preview Screenshot



Aeon is a standalone HTML template designed to help you sell your domains. It comes with both HTML and PHP versions for easier handling of the contact form, Google Analytics, and Google reCAPTCHA.

Choose from 8 different colors for the theme and background, including options for a color background or a textured one.

Template is fully responsive and meets Section 508 / WCAG 2.0 A accessibility standards.




PHP & HTML Versionsindex.php and index.html are both available for your customization needs. The PHP file is fully scripted, while the HTML version provides a semi-functional shell.
PHP Mailer / ConfigSimply edit the “Site-Config.php” file included with the template to get your site off the ground.
Responsive DesignWorks in any size via internal liquid design and media queries.
Well Commented HTMLEach section is split by comments and is easy to read and modify.
Domain List CapableUpload / link to a text list of domains (one per line) to quickly and easily populate additional domains.
reCAPTCHA IntegrationOptionally include your reCAPTCHA keys to add SPAM protection to the contact form. Can also be enabled or disabled at will.
8 ColorsIncludes 8 different color schemes / background combinations to choose from. Thematic CSS is also split out to make it more modifiable.
Texture BackgroundsIncludes two different types of background images, gradient and textured.
PSD Files IncludedFull layered PSD files included for each picture used in the template design.
Google Analytics SupportOptionally fill out the ID code in the “Site-Config.php” file to generate the Google Analytics code in your header.
Cross Browser, IE8+Works in multiple browsers, using CSS2 as a base with CSS3 styles to accent. Includes HTML5 Shim and Placeholders.js support.
Section 508 / WCAG 2.0 ACompliant with US government and accessibility standards.
Font Awesome IconsWe used Font Awesome for all of the template icons, it’s fully integrated an ready to go in the template.



PHP TemplatePHP Version 5+ is recommended.
PHP Contact FormPlease verify that your server can send mail using PHP before uploading.
Google reCAPTCHAcURL Module must be enabled on server for proper checking.



1.Unzip downloaded file and proceed to the “Template” directory.
Modify: Site-Config.php. This sets up domain details, analytics, mailer settings, reCAPTCHA.
Modify: index.php. Make any additional HTML and content modifications in this file.
Mailer: Please make sure your server can handle the PHP mail command.
Upload: Upload a text based list of additional domains (one per line) set the config file.
Theme: Simply change the header Theme CSS link to any of the following colors:
Colors: Blue, Gray, Green, Orange, Purple, Red, Tan, or Teal
Background: Change the class on to the above color of your choice.
2.Upload the contents of the “Template” folder to your server.
3.If not already done, point your domain name at the uploaded files.
4.Install Complete!



[Codester] – Travex – Travel Agency Modern WordPress Theme – Freebies Download

[Codester] – Travex – Travel Agency Modern WordPress Theme – Freebies Download

Travex is Modern, Creative,clean,minimal  WordPress theme. It is  the newest design, super flexible, responsive, working with the awesome Elementor page builder and created with a powerful admin options panel.  It is built for travel agencies, company, tour operators firms, adventurous tours and tourism bureaus services. , Travex includes Elementor so you won’t need to code anything, we can assure you that even if you don’t know anything about coding you will be able to create awesome websites. Travex comes with lots of awesome features, we’ve highlighted some of our main features below, check them to get an idea of how awesome Travex is.

Live Preview


  • Clean, modern,minimal design.
  • Comes with the Elementor Page builder.
  • 2+ home page demo ready.
  • Full Responsive Theme
  • Cross-Browser Compatibility
  • Built with HTML5 and CSS3
  • One Click Demo Import
  • Advanced Customization Themes Options
  • Awesome Slider Options
  • Mega Menu & Retina Ready
  • Contact Form 7 & Mailchimp compatible
  • Unlimited Color Options
  • Advanced Typography Options
  • Lifetime free update and support
  • and much more….


Themeforest – Restaurant Dannys eCommerce WordPress Theme Freebies Download

Themeforest – Restaurant Dannys eCommerce WordPress Theme Freebies Download

Danny’s Restaurant is a clean modern and creative eCommerce WordPress theme for Cafe & Restaurant and any food related business web site witch needs online ordering. Danny’s Restaurant shop is powered by Zion Builder, the powerful Visual Page Builder witch stands at the core of Kallyas WordPress Theme. Either you want to build your website store for your bar, coffee pub, pizza delivery / reservations / booking store, culinary, catering, cuisine, sushi order, wine, cafeteria, classic clean or luxury and modern opentable event, dining, chef, this theme is the right choice for you.

Live Preview

Hogash Power Elite

Restaurant Dannys - 1
Restaurant Dannys - 2
Restaurant Dannys - 3
Restaurant Dannys - 4
Restaurant Dannys - 5

List of features

  • WordPress 4.6+ compatible
  • Bootstrap 3 Framework
  • Amazing frontend LIVE Page Builder containing over 100 elements
  • One-Click Data Install
  • Responsive layout
  • Smart Loader
  • SEO Ready
  • WPML Ready
  • Translation ready
  • Unlimited Colors
  • WooCommerce Integration
  • DummyData Included
  • Unlimited Homepage variations
  • Functional NEWSLETTER – MailChimp integration and adaptation
  • Lots of different Kallyas premium features and plugins
  • Pricing tables
  • Photo Gallery
  • Dynamic PHP Contact form
  • HTML5 & CSS3 Ready
  • PSD Files included (for main demo only)
  • Great social media support!
  • Lots of Built in styles
  • Good Documentation
  • & many others!!


The majority of the images in our demos are stock images from various sources, and they’re not included with the purchase.

Update history:

Version 1.0.5

CodeCanyon – Gusto – Recipes Management System – Freebies Download

CodeCanyon – Gusto – Recipes Management System – Freebies Download

Gusto is a powerful platform to manage recipes for food specific websites. It comes with a plethora of features that make managing the system super easy. Users can register accounts and create their own recipes or you can use the application to manage your own and share them with your friends. The web application is a great way to build your website, very simple to install and customize and we offer great support, That don’t require and experience on web development. Take a look at this wonderful script and you will be impressed even more! Enjoy!





  • Security
    • Totally secured system (SQL injection, XSS, CSRF)
    • Built on powerful CodeIgniter Framework that has been tried and tested by millions of developers.
  • User & Registration System
    • Users can register for an account using the quick and easy register Popup.
    • Login System comes with a Forgotten Password, to allow users to reset their password via email.
  • Recipe Builder
    • Members / Admin add unlimited recipes.
    • Add recipe category, descriptions, Ingredients , video, unlimited multi upload images, featured image, cooking time, people serves, and more …..
    • SEO optimized for recipes using meta keywords and meta tags.
    • Allow anyone to share recipe on social media (Facebook, twitter, google+, ..)
    • Comments on it using facebook plugin.
  • Members
    • Any register user can login/ forget password.
    • Having awesome profile include member information and his recipes.
    • Mange or update his information in profile settings (country, name, username, gender, user profile, …).
    • CRUD (Create, Read, Update, Delete) recipe with more details(featured image, multi upload unlimited images for recipes, description, video,Ingredients, an more amazing details.. ).
    • Comment on other user recipes.
    • Search on website with category or recipe name.
    • Share recipe on social media (Facebook, twitter, google+, ..).
    • See other other members profile.
    • Can follow the new recipes added by register on website subscribe form.
    • Read, share, and comment (with facebook plugin) on blog posts.
    • Contact with support using contact form and contact information.
  • Admin Panel
    • Flexible control panel system with full options.
    • Only admin role can login admin panel.
    • Change website settings(name, logo, favicon, timezone, site description,language, direction…).
    • Change website SEO settings (meta keywords, meta description).
    • Change Contact information (email, phone,…).
    • Unlimited slides images related to the recipe.
    • Change social media pages.
    • Manage Members (Edit, Delete, Add and view member information) and change their member role (Regular member, admin).
    • Manage recipe categories (add, edit, delete, view).
    • Mange recipes (Add, Edit, Delete, and approve member recipe).
    • Approve member recipes to appear on website.
    • Can able/ disable recipe appearing.
    • Mange Blog Posts (add, edit, delete, view).
    • Manage ADS and select its location dynamically otherwise google ads or custom ads images on website.
    • Can view/ delete users subscribed.
    • Manage sliders that appears on website (add, edit, delete, view).
    • Totally with this admin panel can manage and control every thing appear in website with full option.
  • Trendy website interface included.
    • Home page with
      • Dynamic slider images.
      • Widgets for popular recipes, popular authors, counter of pages of social media and simple search
      • latest recipes added
      • Ajax Subscribe form
    • Single page for recipes category included its category and advanced search.
    • Single awesome page for recipes included plethora details.
    • User Profile included all details about author and his recipes.
    • Tips and tricks page include admin posts
    • single page for trip and trick.
    • Contact form to make your client reach you easily.
    • Login/ register/ forget password modal.
    • Amazing Dashboard for register user include setting page, user recipes and page for managing recipe.
  • Other Features
    • DISQUS tool integrated to allow you comment on article. Read documentation to now how to add it in your website
    • Translation is ready for Languages (English and Arabic)
    • RTL Support for admin panel and website
    • Easy to install and usability(without any coding knowledge needed).
    • Awesome Design for Website and admin panel
    • Fully Responsive for Website and admin panel
    • Clean Code with hight performance.
    • Easy Customization
    • Friendly with SEO.
    • High performance.
    • More features will coming up.


  • Backend Framework: Built on CodeIgniter
  • Frontend Framework: Built on Bootstrap
  • Requires PHP 7
  • Supports MySQL, Mysqli.
  • Supports SMTP Email, PHP Mail()


  1. Code Editing Software (eg: Dreamweaver, Sublime Text or Notepad
  2. Web Browser for testing (eg: Google Chrome or Mozilla Firefox)
  3. FTP Tool to upload files to Server (eg: FileZilla).
  4. Read documentation carefully to guide you how to install application.


Version 3.0 Released (10 June 2019)